Closed srhinos closed 2 years ago
Noticed CICD was bypassing the Flake8 file I added since a lot of the CLI args were hardcoded so fixed that.
Also wanted to make sure we fail any contributions that introduce new flake8 violations or don't blacken their code since thats a pretty ezpz step.
Gonna probably also introduce a reqs for dev instead of hard coding them in CICD as well but in another PR!
Ah forgot to mention, Windows was fucked up since my VSCode config was synced from my MacOS work pc and wasn't properly blackening for some reason. That got fixed though so this also includes the ACTUAL blackening
Noticed CICD was bypassing the Flake8 file I added since a lot of the CLI args were hardcoded so fixed that.
Also wanted to make sure we fail any contributions that introduce new flake8 violations or don't blacken their code since thats a pretty ezpz step.
Gonna probably also introduce a reqs for dev instead of hard coding them in CICD as well but in another PR!