Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: pipe_analysis
-
Labels:None
-
Story Points:5
-
Epic Link:
-
Sprint:DRP F20-3 (Aug)
-
Team:Data Release Production
-
Urgent?:No
Description
We have yet to enable a flake8 travis job and branch protections for this repo. As such, our code is becoming very flake8 un-compliant. This ticket is to make all of pipe_analysis flake8 compliant while allowing for the Rubin Style Guide exceptions to PEP 8 , and then turn on the flake8 travis job and branch protections so that no further violations will be able to sneak their way in.
I can confirm that the Travis check is indeed now firing. Here is what I got when I pushed an in-progress (i.e. not yet fully PEP8-ified) branch:
Tim Jenness, are there any additional steps that need to be taken to "turn on" branch protection?