Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enabling stickler-ci #2334

Closed
pelson opened this issue Feb 1, 2017 · 1 comment
Closed

Enabling stickler-ci #2334

pelson opened this issue Feb 1, 2017 · 1 comment
Assignees
@ajdawson @bjlittle
Labels
Experience: Low Status: Decision Required

Comments

@pelson
Copy link
Member

pelson commented Feb 1, 2017

We should cease running pep8 as a test, and start having it as a separate CI process. Stickler-CI looks to be the perfect tool for this. (tested on SciTools/python-stratify#6 if you want to see how it looks).

Ping @bjlittle - I'm minded to go ahead and do this (easy to add) and experiment from there. Happy with that approach?
@ajdawson
Copy link
Member

Agreed, distinguishing genuine test failures from white-space problems would be useful...

@pelson pelson mentioned this issue Oct 31, 2017
Merged
@pelson pelson mentioned this issue Feb 14, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ajdawson ajdawson

@bjlittle bjlittle

Labels
Experience: Low Status: Decision Required
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ajdawson @pelson @bjlittle