Set up continuous testing and pre-commit.

OpenSourceEconomics / skillmodels

MIT License

11 stars 5 forks source link

Set up continuous testing and pre-commit. #37

Closed tobiasraabe closed 5 years ago

tobiasraabe commented 5 years ago

This PR adds continuous integration and pre-commit-hooks to skillmodels.

[ ] Add a badge for Azure and readthedocs to the README. Same as here

tobiasraabe commented 5 years ago

To solve the linting issues, go into the repo folder, create the conda environment on your machine with conda create env, activate the environment with conda activate skillmodels, install pre-commit with pre-commit install and then run the tests with pre-commit run -a. There will be a long list with issues in your terminal window. Most of the errors are easy to understand. For others you might need to consult @janosg.

tobiasraabe commented 5 years ago

@janosg Windows 3.7 works again. I also added flake8-unused-arguments which I find quite useful. Check it out: https://github.com/nhoad/flake8-unused-arguments.