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

docs: simplify commit instruction #346

Merged

Conversation

charles-cooper
Copy link
Member

prefer git commit without -m as it forces you to think about the commit message. git add . is also a bit dangerous since you can accidentally add files you did not mean to add. git commit -a rather only adds changes tracked files.

What I did

How I did it

How to verify it

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

prefer `git commit` without `-m` as it forces you to think about the
commit message. `git add .` is also a bit dangerous since you can
accidentally add files you did not mean to add. `git commit -a` rather
only adds changes tracked files.
@PatrickAlphaC
Copy link
Contributor

LGTM

@charles-cooper charles-cooper merged commit a843212 into vyperlang:master Nov 22, 2024
9 checks passed
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.

3 participants