Contributions are welcome, and they are greatly appreciated!
-
Fork the
whisk
repo on GitHub. -
Clone your fork locally:
$ git clone git@github.com:your_name_here/whisk.git
-
Install your local copy into a venv:
$ cd whisk/ $ python3 -m venv venv $ source venv/bin/activate $ python setup.py develop $ pip install -r requirements_dev.txt
-
Create a branch for local development::
$ git checkout -b name-of-your-bugfix-or-feature
Now you can make your changes locally.
-
When you're done making changes, check that your changes pass flake8 and the tests, including testing other Python versions with tox::
$ flake8 whisk tests $ pytest -s --ignore=whisk/template $ tox
-
Commit your changes and push your branch to GitHub::
$ git add . $ git commit -m "Your detailed description of your changes." $ git push origin name-of-your-bugfix-or-feature
-
Submit a pull request through the GitHub website.
Before you submit a pull request, check that it meets these guidelines:
- The pull request should include tests.
- If the pull request adds functionality, the docs should be updated. Put your new functionality into a function with a docstring.
- The pull request should work for Python 3.6, 3.7 and 3.8, and for PyPy. Check CircleCI and make sure that the tests pass for all Python versions.
Only whisk project collaborators can release a new version.
Make sure all your changes are pushed (including an entry in CHANGELOG.md
) and pass CI tests.
Then run:
$ bump2version patch # possible: major / minor / patch
$ git push
$ git push --tags
$ make release
We intended to have CircleCI perform the release, but it's having issues with git tags.
If doing a patch, you can just run:
$ make bump-push release
To create a demo project from your local fork of whisk:
- Set the following environment variables:
PROJECT_DEMO_DIR=/path/to/dir # The directory where the demo project will be created PROJECT_DEMO_NAME=demo # The name of the project
- Then run the make task:
$ make create-demo
make create-demo
deletes the existing demo project (if it exists) and creates a new project.
To run a subset of tests::
$ pytest tests/test_whisk.py
When running pytest
, you are testing the source code in the current venv
. When running tox
, you are testing the package generated by python setup.py sdist
. It's important to run tox
as it runs test against the package other users will install. tox
can fail even if pytest
succeeds because of an incorrect MANIFEST.in
file or missing dependencies within the setup.py
install_requires
argument.
It's easy to add files to version control but forget to include in the MANIFEST.in file. After committing changes, run the following to see if any files are missing:
$ check-manifest
The project contains a notebook to help orientate new users. You can modify this notebook in the demo project and update the template with:
$ make update-notebook
The following is required to run the included CircleCI workflows:
- An org context named
whisk
. - The following environment variables added to the context:
- GIT_AUTHOR_EMAIL
- GIT_AUTHOR_NAME
- GIT_COMMITTER_EMAIL
- GIT_COMMITTER_NAME
- PYPI_PASSWORD
- PYPI_USERNAME