Skip to content
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

Staging support #44

Merged
merged 4 commits into from
Aug 5, 2022
Merged

Staging support #44

merged 4 commits into from
Aug 5, 2022

Conversation

andrewshadura
Copy link
Owner

  • Clarify the help text (c is for committing, not confirming)
  • Make s actually call stage function
  • Fix staging

@andrewshadura
Copy link
Owner Author

What’s remaining for this is some sort of a test.

The stage hotkey used to only set 'commit' option which was never
consulted, so the stage function was never called.

The stage function used to call git add with -N which only created
and entry in the index but didn’t stage the file itself.

When hitting 's' in commit mode and 'c' in staging mode, ask for
confirmation since it could be the user pressed the wrong key.

Fixes: #41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unintuitive (buggy?) staging behavior / inconsistent documentation
1 participant