Skip to content

Add --no-limit option#204

Merged
tummychow merged 1 commit intotummychow:masterfrom
p1k0chu:118-root-option
Jan 23, 2026
Merged

Add --no-limit option#204
tummychow merged 1 commit intotummychow:masterfrom
p1k0chu:118-root-option

Conversation

@p1k0chu
Copy link
Copy Markdown
Contributor

@p1k0chu p1k0chu commented Jan 22, 2026

disables stack limit,
so revwalk reaches first merge commit, or other author, or actual root

this is basically --root, but isn't named --root because --no-limit doesn't automatically mean your absorb base is the root commit because you can't absorb through a merge,
or other author without the --force-author flag

but it means that absorb stack will go as far as possible, allowing linear histories to even reach real root commit

should be used carefully in case theres like 100k commits without any merges and from a single author...

Closes: #118

@p1k0chu p1k0chu marked this pull request as draft January 22, 2026 08:23
@p1k0chu
Copy link
Copy Markdown
Contributor Author

p1k0chu commented Jan 22, 2026

fixed tests (that i broke) + added unit test for this

@p1k0chu p1k0chu marked this pull request as ready for review January 22, 2026 08:36
@blairconrad
Copy link
Copy Markdown
Contributor

blairconrad commented Jan 22, 2026

This change does not include docs update.

disables stack limit,
so revwalk reaches first merge commit, or other author, or actual root

this is basically --root, but isn't named --root because
--no-limit doesn't automatically mean your absorb base is the root commit
because you can't absorb through a merge,
or other author without the --force-author flag

but it means that absorb stack will go as far as possible,
allowing linear histories to even reach real root commit

should be used carefully in case theres like 100k commits
without any merges and from a single author...

Closes: tummychow#118
Signed-off-by: p1k0chu <p1k0chu.vim@gmail.com>
@tummychow tummychow merged commit ddda0b7 into tummychow:master Jan 23, 2026
7 checks passed
@p1k0chu p1k0chu deleted the 118-root-option branch January 23, 2026 04:26
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.

Add --root option

3 participants