Skip to content
This repository has been archived by the owner on Apr 1, 2020. It is now read-only.

Add free automated flake8 testing on all pull requests #6

Merged
merged 1 commit into from
Aug 24, 2017

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Aug 24, 2017

Use Travis CI to do automated continuous integration -- in this case flake8 testing. The owner of the this repo would need to go to https://travis-ci.org/profile and flip the repository switch on to enable free automated flake8 testing on each pull request.

Use Travis CI to do automated continuous integration -- in this case flake8 testing.  The owner of the this repo would need to go to https://travis-ci.org/profile and flip the repository switch on to enable free automated flake8 testing on each pull request.
@zrzka
Copy link
Owner

zrzka commented Aug 24, 2017

Was thinking about this as well, but you were faster :) Thanks!

@zrzka zrzka merged commit 112d4ca into zrzka:master Aug 24, 2017
@cclauss cclauss deleted the patch-1 branch August 24, 2017 09:26
@cclauss
Copy link
Contributor Author

cclauss commented Aug 24, 2017

The second pass can be a bit picky https://travis-ci.org/zrzka/blackmamba#L459 but you can always --ignore=Ennn,Fnnn any issues that you do not want flagged. Repos like CPython and Django have zero tolerance. They just do flake8 .

@zrzka
Copy link
Owner

zrzka commented Aug 24, 2017

I do personally like zero tolerance and treat warnings as errors in other projects.

@zrzka
Copy link
Owner

zrzka commented Aug 24, 2017

Down to 0, thanks again.

screen shot 2017-08-24 at 12 09 02

@cclauss
Copy link
Contributor Author

cclauss commented Aug 24, 2017

If you want you can remove the time prefix to the 2 flake8 commands... I was just trying to see how fast flake8 was.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants