Bump pipeline dependencies #1250
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The CI also yells at us for not updating mdbook submodules, but from what I can recall you telling me, the devs have a reputation for making breaking changes? 
|
|
The typechecking CI passes, so that looks good, and the only way to test deployment is to merge this, so go for it. There were actual breaking changes that affect us in newer versions of mdbook, which IIRC some of the newer modules depend on, so I'd separate updating actions from updating mdbook. |
VojtechStep
approved these changes
Jan 30, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Due to deprecations with the GitHub API, some of our pipeline dependencies have started failing today, and more will tomorrow. I'm taking the opportunity do to some midwinter cleaning and updating all of our dependencies. Here's the homework I've done, from most to least important:
actions/upload-pages-artifact@v3updates toupload-artifact@v4(broke today) and removes a builtinchmodcommand we don't useactions/deploy-pages@v4updates toupload-artifact@v4(broke today), updates to Node20actions/cargo-install@v>=3.2update to new GitHub API (otherwise breaks tomorrow)actions/configure-pages@v5swaps to Next.js 13.3 and newer, updates to Node20actions/cache@v4updates to Node20actions/checkout@v4updates to Node20actions/actions-mdbook@v3updates to Node20actions/setup-python@v5updates to Node20pre-commit/[email protected]updates dependency toactions/cache@v4Some input parameters have also been deprecated and removed, I don't know if this will affect us yet.Since there are no warnings in our CI logs that we are using deprecated input parameters, I'm assuming we're good on this front.