New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: pre-commit infrastructure and update travis #243
Conversation
70fdfae
to
3ecdab5
Compare
Please do check out the Travis build for the new test matrix: https://travis-ci.org/github/autoprotocol/autoprotocol-python/builds/689570208 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
neat! thanks for simplifying things.
overall:
- pre-commit is now the primary source of truth for what's to be run
- tox is a small virtualenv wrapper around it
- travis calls individual tox steps directly
- python: '3.6' | ||
env: TOXENV=lint | ||
- python: '3.6' | ||
env: TOXENV=clean,py36,stats |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is the clean step here necessary still?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, yea, its to blow away coverage artifacts if present. It seems common in other travis integration libraries, so I assume its doing something but haven't validated
somewhat related, can we configure travis so that it's using something newer than xenial? |
yea, could use Bionic (https://docs.travis-ci.com/user/reference/linux/). It shouldn't really matter since we're using mostly pip/tox for the running, so it should be fine to upgrade. I'll try it |
Add pre-commit infrastructure to help with basic lint tasks. This is in preparation of adding an autoformatter for the repo. As part of this, the travis file is reorganized to help with clearer delineation of jobs. Documention is also updated, with the Contributing section updated with more explicit testing steps and the new environment and lint setup. This includes all the fixes from the pre-commit hooks added.
Add pre-commit infrastructure to help with basic lint tasks. This is in
preparation of adding an autoformatter for the repo.
As part of this, the travis file is reorganized to help with clearer
delineation of jobs.
Documention is also updated, with the Contributing section updated with
more explicit testing steps and the new environment and lint setup.
This includes all the fixes from the pre-commit hooks added.