Skip to content
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 workflow to update the docker requirements #287

Merged
merged 5 commits into from
Jan 22, 2024

Conversation

pmeier
Copy link
Member

@pmeier pmeier commented Jan 22, 2024

Follow-up to #283 to automatically update the requirements. Basically, we only need to dispatch the workflow and a bot will open a PR for us.

@pmeier pmeier requested a review from nenb January 22, 2024 09:40
@pmeier
Copy link
Member Author

pmeier commented Jan 22, 2024

Failing CI is expected here, because we added a new dependency in #281 that is not yet reflected in the requirements. I'll dispatch a workflow to update the requirements when this PR is merged.

Copy link
Contributor

@nenb nenb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@pmeier pmeier merged commit 00e581f into main Jan 22, 2024
10 of 11 checks passed
@pmeier pmeier deleted the docker-requirements-update branch January 22, 2024 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants