Skip to content

Fix workflow syntax warning and update checkout to Node.js 24#17

Merged
picatz merged 1 commit intomainfrom
fix-warnings
Mar 23, 2026
Merged

Fix workflow syntax warning and update checkout to Node.js 24#17
picatz merged 1 commit intomainfrom
fix-warnings

Conversation

@picatz
Copy link
Copy Markdown
Contributor

@picatz picatz commented Mar 23, 2026

What was changed

Wrap the entire if condition in ${{ }}, and update actions/checkout from v4 to v6.

Why?

To resolve Node.js 20 deprecation and expression syntax warnings:

Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4.
Workflow syntax warning: .github/workflows/require-codeowners.yml#L35
                                                                                                                                     
.github/workflows/require-codeowners.yml (Line: 35, Col: 9): Conditional expression contains literal text outside replacement tokens. This will cause the expression to always evaluate to truthy. Did you mean to put the entire expression inside ${{ }}?

Wrap the entire `if` condition in ${{ }} to fix the literal text warning,
and update actions/checkout from v4 to v6 to resolve Node.js 20 deprecation.
@picatz picatz requested a review from a team as a code owner March 23, 2026 17:35
@picatz picatz merged commit a36114c into main Mar 23, 2026
3 checks passed
@picatz picatz deleted the fix-warnings branch March 23, 2026 17:38
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