Browse Source

Merge pull request #2305 from sorech02/updating_contributing_rst

Updating contributing rst
pull/921/merge
David Lord 8 years ago committed by GitHub
parent
commit
54a18a5e94
  1. 43
      CONTRIBUTING.rst

43
CONTRIBUTING.rst

@ -24,13 +24,42 @@ Reporting issues
Submitting patches Submitting patches
================== ==================
- Include tests if your patch is supposed to solve a bug, and explain First time setup
clearly under which circumstances the bug happens. Make sure the test fails ----------------
without your patch.
- Download and install the `latest version of git`_.
- Try to follow `PEP8 <https://www.python.org/dev/peps/pep-0008/>`_, but you - Configure git with your `username`_ and `email`_.
may ignore the line-length-limit if following it would make the code uglier. - Make sure you have a `GitHub account`_.
- Fork Flask to your GitHub account by clicking the `Fork`_ button.
- `Clone`_ your GitHub fork locally.
- Add the main repository as a remote to update later.
``git remote add pallets https://github.com/pallets/flask``
.. _GitHub account: https://github.com/join
.. _latest version of git: https://git-scm.com/downloads
.. _username: https://help.github.com/articles/setting-your-username-in-git/
.. _email: https://help.github.com/articles/setting-your-email-in-git/
.. _Fork: https://github.com/pallets/flask/pull/2305#fork-destination-box
.. _Clone: https://help.github.com/articles/fork-a-repo/#step-2-create-a-local-clone-of-your-fork
Start coding
------------
- Create a branch to identify the issue you would like to work on (e.g.
``2287-dry-test-suite``)
- Using your favorite editor, make your changes, `committing as you go`_.
- Try to follow `PEP8`_, but you may ignore the line length limit if following
it would make the code uglier.
- Include tests that cover any code changes you make. Make sure the test fails
without your patch. `Run the tests. <contributing-testsuite_>`_.
- Push your commits to GitHub and `create a pull request`_.
- Celebrate 🎉
.. _committing as you go: http://dont-be-afraid-to-commit.readthedocs.io/en/latest/git/commandlinegit.html#commit-your-changes
.. _PEP8: https://pep8.org/
.. _create a pull request: https://help.github.com/articles/creating-a-pull-request/
.. _contributing-testsuite:
Running the testsuite Running the testsuite
--------------------- ---------------------

Loading…
Cancel
Save