Skip to content

Setup latest node in doc build ci#2846

Merged
JordonPhillips merged 1 commit intomainfrom
explicit-node-setup
Nov 12, 2025
Merged

Setup latest node in doc build ci#2846
JordonPhillips merged 1 commit intomainfrom
explicit-node-setup

Conversation

@JordonPhillips
Copy link
Contributor

This updates the doc build part of ci to install the latest version of node rather than relying on whatever version happens to be around.

This also adds caching to the make-docs workflow action.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@JordonPhillips JordonPhillips requested a review from a team as a code owner November 12, 2025 15:25
@JordonPhillips
Copy link
Contributor Author

Note the changelog issue is resolved by #2847

This updates the doc build part of ci to install the latest version
of node rather than relying on whatever version happens to be around.

This also adds caching to the make-docs workflow action.
@github-actions
Copy link
Contributor

This pull request does not contain a staged changelog entry. To create one, use the ./.changes/new-change command. For example:

./.changes/new-change --pull-requests "#2846" --type feature --description "Setup latest node in doc build ci"

Make sure that the description is appropriate for a changelog entry and that the proper feature type is used. See ./.changes/README or run ./.changes/new-change -h for more information.

@JordonPhillips JordonPhillips added the no-changelog-needed This PR doesn't require a changelog entry, so the changelog automation tool should not run on it. label Nov 12, 2025
@JordonPhillips JordonPhillips merged commit 47b8db0 into main Nov 12, 2025
13 checks passed
@JordonPhillips JordonPhillips deleted the explicit-node-setup branch November 12, 2025 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog-needed This PR doesn't require a changelog entry, so the changelog automation tool should not run on it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants