-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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.
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
flake8 updates #5767
flake8 updates #5767
Conversation
This pull request fixes 4 alerts when merging 89dcac39573b7e76dd9f47ff17717af1f993f156 into 622b8b5 - view on LGTM.com fixed alerts:
|
Codecov Report
@@ Coverage Diff @@
## master #5767 +/- ##
=======================================
Coverage 58.20% 58.20%
=======================================
Files 172 172
Lines 19461 19461
Branches 3209 3209
=======================================
Hits 11327 11327
Misses 7118 7118
Partials 1016 1016 |
This pull request fixes 4 alerts when merging ec3c86c2ed0291aabc5d97d73035b45db060afc0 into 216f7da - view on LGTM.com fixed alerts:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Failing SPC GeoNode local tests
ec3c86c
to
80a07af
Compare
This pull request fixes 4 alerts when merging 80a07af6233e744d04279b81e2a261ebab88e5b8 into c241909 - view on LGTM.com fixed alerts:
|
@travislbrundage Just for your information: #5720 |
I made some updates but I am not sure how to run the spcgeonode local tests, sorry if I am missing something obvious. Does that just mean the docker doesn't build properly? I adjusted my changes to preserve the way things were done before in the spcgeonode file, so it should be functionally identical now. Updating to fix the file conflicts now too. |
hi @travislbrundage docs have been removed from master see: #5629 |
This pull request fixes 4 alerts when merging 8d6e681 into 690c446 - view on LGTM.com fixed alerts:
|
Fixing remaining files which had flake8 complaints.
Checklist
For all pull requests:
The following are required only for core and extension modules (they are welcomed, but not required, for contrib modules):
Submitting the PR does not require you to check all items, but by the time it gets merged, they should be either satisfied or inapplicable.