-
Notifications
You must be signed in to change notification settings - Fork 12
Ignore -m option when used
#28
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
Conversation
3d1c0e1 to
f7feea4
Compare
sbimochan
commented
Nov 1, 2024
Do you also mistakenly add 'git' in front of 'commit -m' as well? Haha!
What do others say about this graceful fallback?
One con I see is it's adding complexity to code for other future contributors because this script doesn't have any tests.
sbimochan
commented
Nov 1, 2024
Meanwhile I'd recommend to use your own fork. I'm unsure how many percentile of the users have this issue.
I'll be adding full tests and setups before merging this to main. Or contributions are of-course welcome. 😀
sbimochan
commented
Nov 1, 2024
Ref #29
sbimochan
commented
Nov 4, 2024
@maskeynihal please provide tests to cover your changes here
commit "<commit message>"commit -m "<commit message>"These two will both gives the same output.
-mis ignore.Motivation
- The
image-moption is typically used with thegit commit -mcommand. However, sometimes out of habit, users may mistakenly add-mwhen using thecommitcommand alone, resulting in an error, as illustrated below.