-
Notifications
You must be signed in to change notification settings - Fork 17
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
[WIP] add docker worker #374
base: master
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## master #374 +/- ##
==========================================
- Coverage 93.36% 92.83% -0.54%
==========================================
Files 99 95 -4
Lines 8248 7549 -699
==========================================
- Hits 7701 7008 -693
+ Misses 547 541 -6
Continue to review full report at Codecov.
|
Hello @glemaitre! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2020-10-16 13:54:15 UTC |
This pull request introduces 6 alerts and fixes 3 when merging 5534c9d into a13643d - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 5 alerts and fixes 3 when merging cf56bf0 into c7a3562 - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 2 alerts when merging 8072292 into 7e975d0 - view on LGTM.com new alerts:
|
closes #290