Skip to content

New Rule: Git commit with no added files #1131

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.

Sign up for GitHub

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

Jump to bottom
Merged
scorphus merged 2 commits into from
Jul 15, 2021
Merged

New Rule: Git commit with no added files #1131

scorphus merged 2 commits into from
Jul 15, 2021

Conversation

@the-docta
Copy link
Contributor

@the-docta the-docta commented Sep 8, 2020

if "git commit" is executed with no staged changes, this will add -a to the command.
e.g.

  • git commit => git commit -a
  • git commit -m "blubb" => git commit -a -m "blubb"

@the-docta the-docta changed the title Git commit with no added files New Rule: Git commit with no added files Sep 10, 2020
@scorphus
nvbn#1131: Improve git_commit_add rule
11b7052 
Add more capabilities to the rule, remove its priority and fix tests
@scorphus scorphus force-pushed the git_commit_add branch 2 times, most recently from 3a0ff25 to 92f70bb Compare July 8, 2021 19:56
scorphus pushed a commit to the-docta/thefuck that referenced this pull request Jul 15, 2021
@the-docta @scorphus
nvbn#1131: Add rule for Git commit with no added files
5ffee36
scorphus added a commit to the-docta/thefuck that referenced this pull request Jul 15, 2021
@scorphus
nvbn#1131: Improve git_commit_add rule
ea97d27 
Add more capabilities to the rule, remove its priority and fix tests
@scorphus scorphus force-pushed the git_commit_add branch 2 times, most recently from ea97d27 to 11b7052 Compare July 15, 2021 11:58
scorphus
scorphus approved these changes Jul 15, 2021
View reviewed changes
Copy link
Collaborator

@scorphus scorphus left a comment

Choose a reason for hiding this comment

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

Thanks for contributing, @the-docta. Please check the 2nd commit in this PR 🙂

@scorphus scorphus merged commit 11b7052 into nvbn:master Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@scorphus scorphus scorphus approved these changes

+1 more reviewer

@dvjn dvjn dvjn approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@the-docta @scorphus @dvjn