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

Bump pipeline to v2024.10.11.170331 #4666

Merged
merged 2 commits into from
Oct 16, 2024
Merged

Bump pipeline to v2024.10.11.170331 #4666

merged 2 commits into from
Oct 16, 2024

Conversation

alarthast
Copy link
Contributor

@alarthast alarthast commented Oct 11, 2024

Fixes #4661.

@evansd
Copy link
Contributor

evansd commented Oct 14, 2024

There's an additional change that we'll need to make here which is that the opensafely-pipeline dependency in requirements.prod.in needs to change to:

opensafely-pipeline[fastparser]@...

This will pull in the necessary dependencies to give us fast YAML parsing. More details in the pipeline PR:

@alarthast alarthast force-pushed the bump-pipeline branch 2 times, most recently from df0ba66 to 18c2401 Compare October 14, 2024 11:05
@alarthast alarthast marked this pull request as ready for review October 14, 2024 11:06
@alarthast alarthast changed the title Bump pipeline Bump pipeline to v2024.10.11.170331 Oct 14, 2024
Copy link
Contributor

@StevenMaude StevenMaude left a comment

Choose a reason for hiding this comment

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

Looks good; thanks!

@alarthast alarthast merged commit 283b6eb into main Oct 16, 2024
7 checks passed
@alarthast alarthast deleted the bump-pipeline branch October 16, 2024 11:50
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.

Remove redundant dependencies after removing L4 file viewer code
3 participants