-
Notifications
You must be signed in to change notification settings - Fork 35
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
Fixing the nightly job #290
Conversation
WARNING!!! This PR is not attached to an issue. In most cases this is not advisable. Please see our PR docs for more information about how to attach this PR to an issue. |
This is not quite right. We need to somehow split the test steps that use the matrix from the other steps (publish docs, clients, etc). Edit: Filed issue https://pulp.plan.io/issues/7874 |
from L70 it needs to be another job that depends on the previous one |
359b1c3
to
5f8939c
Compare
needs: test | ||
|
||
env: | ||
- TEST: docs |
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.
some steps need TEST=bindings
, maybe we can set it by step
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.
It looks like some steps need TEST=bindings
while others need TEST=docs
. I need to either:
- Break these steps up into jobs each with their own before_install/install/..
- Rewrite the scripts. Maybe add a
TEST=publish
or something?
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.
I don't have a strong opinion on that, I would pick the easier/faster way, or maybe move the docs to publish docs workflow?
501eebc
to
e6dcf5f
Compare
[noissue]
e6dcf5f
to
4af22d9
Compare
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.
Awesome!
[noissue]