-
Notifications
You must be signed in to change notification settings - Fork 20
Conversation
d4f032c
to
13000c3
Compare
Codecov Report
@@ Coverage Diff @@
## master #59 +/- ##
=======================================
Coverage 42.53% 42.53%
=======================================
Files 22 22
Lines 710 710
=======================================
Hits 302 302
Misses 408 408 Continue to review full report at Codecov.
|
05dfaae
to
d695fcd
Compare
I think a separate job would be almost as efficient and a little bit clearer. |
d695fcd
to
d8a0a44
Compare
Note that I am not creating a separate job for testing the docs. We're limited by the number of concurrent jobs in the Pulp org so I figure it's more efficient to execute the docs check during the normal test since it adds 30-60 seconds. fixes #4403 https://pulp.plan.io/issues/4403
d8a0a44
to
d6196a5
Compare
@@ -7,10 +7,12 @@ python: | |||
- "3.6" | |||
- "3.7" | |||
env: | |||
matrix: | |||
- DJANGO_MAX=2.2.100 DB=postgres TEST=pulp |
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.
Not sure if we need this DJANGO_MAX environment variable? I can add it back if we do.
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.
We can add it back later.
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.
This looks like what we need! Thanks @daviddavis 👍
Note that I am not creating a separate job for testing the docs. We're
limited by the number of concurrent jobs in the Pulp org so I figure
it's more efficient to execute the docs check during the normal test
since it only adds 30-60 seconds.
fixes #4403
https://pulp.plan.io/issues/4403