Skip to content

Instantly share code, notes, and snippets.

@linjunpop
Last active October 22, 2021 22:15
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save linjunpop/4591629 to your computer and use it in GitHub Desktop.
Save linjunpop/4591629 to your computer and use it in GitHub Desktop.
Github the correct way.

Use Github, the correct way, IMO

  1. DO NOT commit anything on master, except wording change.
  2. Write descriptive commit messages. (The reason for this change.)
  3. Always use $ git pull --rebase.
  4. DO NOT merge PR raised by yourself.
  5. Close related issue after merge in PR. (Closes #123)
  6. Write down everything in github comments, include links, codes, etc.
  7. Let all tests pass before merge any PR. (CI server FTW)
  8. Add link to related issue in the PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment