feat(commands): add cz commit arg to automatically stage every tracked and un-staged file

[Xiao75896453]

Description

Adding cz commit arg to automatically stage every tracked and un-staged file

Checklist

• Add test cases to all the changes you introduce
• Run ./scripts/format and ./scripts/test locally to ensure this change passes linter check and test
• Test the changes on the local machine manually
• Update the documentation for the changes

Expected behavior

command cz commit -a will automatically stage every tracked and un-staged file .

Steps to Test This Pull Request

1. cz commit -a

Additional context

#554

[codecov]

Codecov Report

Patch coverage is 97.51% of modified lines.

Files Changed	Coverage
commitizen/cli.py	ø
commitizen/git.py	33.33%
commitizen/providers/scm_provider.py	91.17%
commitizen/__version__.py	100.00%
commitizen/commands/bump.py	100.00%
commitizen/commands/commit.py	100.00%
commitizen/defaults.py	100.00%
commitizen/providers/__init__.py	100.00%
commitizen/providers/base_provider.py	100.00%
commitizen/providers/cargo_provider.py	100.00%
... and 5 more

📢 Thoughts on this report? Let us know!.

[woile]

I don't think this is what -a does. In this case even non-tracked files are being added. -a should happen on git commit -a

[Xiao75896453]

Thank you! I misunderstood what git commit -a usage. I will modify this PR.

[Lee-W]

ah yes, I think I misunderstood during the sprint as well 🫠 I belive you can check how the signoff feature is implemented. They are supposed to be similar

[woile]

I wonder what's the advantage of doing git add -u vs doing git commit -a?
The plus side of git commit -a is that we call git once, does git add give some kind of different ergonomics?

[Xiao75896453]

I do git add -u instead of git commit -a because of if git.is_staging_clean() and not dry_run.
If doing git commit -a, I think I may need to modify if git.is_staging_clean() and not dry_run and handle the simultaneous existence of git commit -s and git commit -a.
I feel it is more intuitive to use git add -u

[Lee-W]

I do git add -u instead of git commit -a because of if git.is_staging_clean() and not dry_run. If doing git commit -a, I think I may need to modify if git.is_staging_clean() and not dry_run and handle the simultaneous existence of git commit -s and git commit -a. I feel it is more intuitive to use git add -u

yep, agree with your point. @woile what do you think?

[woile]

Sounds good