Skip to content

Latest commit

 

History

History
82 lines (57 loc) · 2.92 KB

CONTRIBUTING.md

File metadata and controls

82 lines (57 loc) · 2.92 KB

Contributing to Speakerfight

We'd love to have your contribution to our project and to make Speakerfight. Here are the guidelines we'd like you to follow:

Issues and Bugs

If you find a bug in the source code or a mistake in the documentation, you can help us by submitting an issue to our GitHub repository. Even better, you can submit a pull request with a fix.

Submission Guidelines

Assigning a issue to yourself

If you want to fix an issue, please comment in the issue thread to show everyone that you want to do that. Ask anything about it, explain your thoughts. It is very important for us to have that conversation in order to avoid misconceptions, bugs, non-necesseracy implementations and over-engineering.

Submitting an Issue

Before you submit your issue, search in the archive, maybe your problem was already answered. Also check if there is no work in progress to fix your problem (when it wasn't created any issue for the problem ).

Submitting a pull request

Build the environment:

$ git clone [email protected]:luanfonceca/speakerfight.git
$ cd speakerfight
$ pip install -r requirements.txt
$ python manage.py migrate # we have a fixture for the users...
$ python manage.py loaddata deck/fixtures/user.json
# username: admin, password: admin;
# username: user, password: user
$ python manage.py loaddata deck/fixtures/event.json
$ python manage.py loaddata deck/fixtures/proposal.json
$ python manage.py loaddata deck/fixtures/socialapp.json
$ python manage.py runserver

Before you submit your pull request consider the following guidelines:

  • Make your changes in a new branch:

     git checkout -b my-fix-branch master
  • Follow our Coding Rules.

  • Run the test suite:

     python manage.py test
  • Run code coverage:

     coverage run manage.py test
  • Commit your work, including appropriate test cases (unit tests, functional tests, regression tests, etc).

  • When commiting remember to write clear commit messages explaining why the change is important.

  • Run the test suite to ensure your code is working properly.

  • Push to the upstream and create a pull request against the master branch.

Coding Rules

To ensure consistency throughout the source code, keep the following rules in mind:

  • Speakerfight follows the same conventions Django uses.
  • Some programming languages have their own coding style guide. EditorConfig is here to help us on this.
  • All features or changes must be tested (your patch should have tests proving your code works as expected).
  • Code coverage is important.