forked from orbit-oss/flask
add more git commands to contributing doc
This commit is contained in:
parent
8609b7d840
commit
c2b4b745e9
1 changed files with 17 additions and 4 deletions
|
|
@ -82,14 +82,27 @@ First time setup
|
||||||
Start coding
|
Start coding
|
||||||
~~~~~~~~~~~~
|
~~~~~~~~~~~~
|
||||||
|
|
||||||
- Create a branch to identify the issue you would like to work on (e.g.
|
- Create a branch to identify the issue you would like to work on::
|
||||||
``2287-dry-test-suite``)
|
|
||||||
- Using your favorite editor, make your changes, `committing as you go`_.
|
git branch your-branch-name
|
||||||
|
|
||||||
|
- Then switch to make sure that we are working on that branch by using::
|
||||||
|
|
||||||
|
git checkout your-branch-name
|
||||||
|
|
||||||
|
- Using your favorite editor, make your changes, `committing as you go`_ by using the following::
|
||||||
|
|
||||||
|
git add -A
|
||||||
|
git commit
|
||||||
|
|
||||||
- Try to follow `PEP8`_, but you may ignore the line length limit if following
|
- Try to follow `PEP8`_, but you may ignore the line length limit if following
|
||||||
it would make the code uglier.
|
it would make the code uglier.
|
||||||
- Include tests that cover any code changes you make. Make sure the test fails
|
- Include tests that cover any code changes you make. Make sure the test fails
|
||||||
without your patch. `Run the tests. <contributing-testsuite_>`_.
|
without your patch. `Run the tests. <contributing-testsuite_>`_.
|
||||||
- Push your commits to GitHub and `create a pull request`_.
|
- Push your commits to GitHub and `create a pull request`_ by using::
|
||||||
|
|
||||||
|
git push --set-upstream origin your-branch-name
|
||||||
|
|
||||||
- Celebrate 🎉
|
- Celebrate 🎉
|
||||||
|
|
||||||
.. _committing as you go: http://dont-be-afraid-to-commit.readthedocs.io/en/latest/git/commandlinegit.html#commit-your-changes
|
.. _committing as you go: http://dont-be-afraid-to-commit.readthedocs.io/en/latest/git/commandlinegit.html#commit-your-changes
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue