Skip to content

Take rebase options from command-line #156

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Mar 7, 2025

Conversation

blairconrad
Copy link
Contributor

Fixes #88.

Assigning an author to begin with slightly simplifies adding test
commits, and will also support workflows that actually run git to create
new commits.
No production code change. Just test the current
behaviour before we change it.
Options will be read from all arguments after `--`,
and will be passed to git rebase as-is.
Throws an error if --and-rebase is not also specified.
@tummychow tummychow merged commit bc179c5 into tummychow:master Mar 7, 2025
5 checks passed
@tummychow
Copy link
Owner

thanks, nice work. unfortunate that we have to chdir during a test. maybe we should be invoking git-rebase with -C... not a blocker for this pr.

are there more things you wanted to send prs for before i cut v0.7.0?

@blairconrad blairconrad deleted the rebase-args branch March 7, 2025 23:51
@blairconrad
Copy link
Contributor Author

blairconrad commented Mar 7, 2025

Oh, I'm just noodling along. The only feature I was interested in was config for one-fixup-per-commit. I have no agenda otherwise aside from "I've learned how to work within the repo, so why not clean up some old issues". If you want to direct me toward something, whether it's feature work or infrastructure, please do, but I'm not interested in holding up a release!

I appreciate you indulging me. I'm having a good time.

@blairconrad
Copy link
Contributor Author

blairconrad commented Mar 8, 2025

Was not aware of git -C. May play. If it means the weird directory-changing can go, I think it'd be worth it.

blairconrad added a commit to blairconrad/git-absorb that referenced this pull request Mar 8, 2025
From [a suggestion made in tummychow#156](tummychow#156 (comment)).

In addition to ensuring we're running from the right directory in
production, makes it easier to write tests that rebase.
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.

Support additional flags when running rebase
2 participants