Skip to content

Conversation

@desrosj
Copy link
Member

@desrosj desrosj commented Aug 6, 2025

What?

When both node-version-file and node-version are passed to actions/setup-node, a warning is output.

When inputs are empty, they are ignored. This will silence the error while keeping the same functionality.

Why?

Avoiding warnings for expected outcomes helps to avoid distractions.

When both `node-version-file` and `node-version` are passed to `actions/setup-node`, a warning is output.

When inputs are empty, they are ignored. This will silence the error while keepign the same functionality.
@desrosj desrosj self-assigned this Aug 6, 2025
@desrosj desrosj marked this pull request as ready for review August 6, 2025 11:33
@desrosj desrosj requested a review from kevin940726 August 6, 2025 11:33
@github-actions
Copy link

github-actions bot commented Aug 6, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: desrosj <[email protected]>
Co-authored-by: johnbillion <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@desrosj desrosj requested a review from johnbillion August 6, 2025 11:33
@desrosj desrosj added the [Type] Build Tooling Issues or PRs related to build tooling label Aug 6, 2025
Copy link
Member

@johnbillion johnbillion left a comment

Choose a reason for hiding this comment

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

@desrosj desrosj merged commit 58a5abc into trunk Aug 14, 2025
66 of 69 checks passed
@desrosj desrosj deleted the try/dynamic-node-version branch August 14, 2025 03:12
@github-actions github-actions bot added this to the Gutenberg 21.5 milestone Aug 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Build Tooling Issues or PRs related to build tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants