-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add one check job for updates on automation/requirements.txt #1102
Add one check job for updates on automation/requirements.txt #1102
Conversation
a29d9e7
to
c2c710f
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.
Would it make sense to make rpm_list
depend on check_resolution
(via needs: check_resolution
)?
Yes, it does make sense to wait on |
c2c710f
to
e4c49bd
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.
Last thought: would this workflow work if you also ran it when .github/workflows/requirement-check.yml
itself changes? Sort of implementing CI for your CI.
For that to happen we need to change this conditional
Let me send the change |
e4c49bd
to
dfa24f9
Compare
Exactly what I had in mind. And we immediately see it fail, so it's "working" :) |
dfa24f9
to
b320221
Compare
I consider this a win, it's working! |
No description provided.