Closed pastorhudson closed 2 years ago
We are not passing flake8 because we have formatting issues. We could correct these or skip flake8. I'm going to remove flake8 since that should be a separate issue / pull request.
This workflow will build and test. We can add other workflows to build docs, release, and publish to pypi using option 2 as described here https://stackoverflow.com/a/65698892
I think this is ready to go.
There are a bunch of linting errors. We can pass by pinning pylint to 2.4.4 for now and then correct the linting errors in a future PR?
Merging #53 (9893e4a) into master (fd6fe4b) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #53 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 5 5
Lines 171 173 +2
=========================================
+ Hits 171 173 +2
Impacted Files | Coverage Δ | |
---|---|---|
pypco/auth_config.py | 100.00% <0.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
I think this basic github action will get testing going.