Skip to content

Add -r/--revisions CLI arg to review a revset/commit range#157

Merged
YPares merged 1 commit intoagavra:mainfrom
YPares:push-mwplqsrrqsvr
Feb 9, 2026
Merged

Add -r/--revisions CLI arg to review a revset/commit range#157
YPares merged 1 commit intoagavra:mainfrom
YPares:push-mwplqsrrqsvr

Conversation

@YPares
Copy link
Copy Markdown
Collaborator

@YPares YPares commented Feb 9, 2026

You can run tuicr with the --revset/-r arg to go straight into reviewing a set of revisions. For instance:

tuicr --revset 'main..'

This automatically starts reviewing all commits which are not in main. Note that if the selected revset is disjoint, intermediary revisions will be included too so we still end up with a valid range without "holes".

This also factorizes App construction.

Copy link
Copy Markdown
Owner

@agavra agavra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Feel free to merge when CI is green.

@YPares YPares force-pushed the push-mwplqsrrqsvr branch from ffc8373 to 795271a Compare February 9, 2026 17:39
@YPares YPares force-pushed the push-mwplqsrrqsvr branch 3 times, most recently from bf58047 to c90f32f Compare February 9, 2026 21:32
@YPares
Copy link
Copy Markdown
Collaborator Author

YPares commented Feb 9, 2026

@naps62 @agavra This has been updated with Git and Mercurial support :) if you care to test it

@YPares YPares force-pushed the push-mwplqsrrqsvr branch from c90f32f to 5da3823 Compare February 9, 2026 21:34
@YPares YPares changed the title Add -r/--revset CLI flag to review a JJ revset as a commit range Add -r/--revisions CLI arg to review a revset/commit range Feb 9, 2026
@YPares YPares requested a review from agavra February 9, 2026 21:38
@YPares YPares force-pushed the push-mwplqsrrqsvr branch from 5da3823 to ce7f42c Compare February 9, 2026 21:39
@agavra
Copy link
Copy Markdown
Owner

agavra commented Feb 9, 2026

Nice, tested locally and more complex expressions like -r main^^..HEAD work as well

@YPares YPares merged commit 8f7b75e into agavra:main Feb 9, 2026
4 checks passed
@YPares YPares deleted the push-mwplqsrrqsvr branch February 9, 2026 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants