Skip to content

chore: update node and actions/core #33

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

blazceltra
Copy link

PR description

Update NodeJS version to 20 (latest supported by GH Actions) and @actions/core package so it no longer display a deprecation warning message bellow (action output is redacted for privacy reasons).

Screenshot 2025-02-07 at 12 05 24

Change in compiled JS

Old setOutput in dist/index.js

function setOutput(name, value) {
    process.stdout.write(os.EOL);
    command_1.issueCommand('set-output', { name }, value);
}
exports.setOutput = setOutput;

New setOuput() in dist/index.js

function setOutput(name, value) {
    const filePath = process.env['GITHUB_OUTPUT'] || '';
    if (filePath) {
        return (0, file_command_1.issueFileCommand)('OUTPUT', (0, file_command_1.prepareKeyValueMessage)(name, value));
    }
    process.stdout.write(os.EOL);
    (0, command_1.issueCommand)('set-output', { name }, (0, utils_1.toCommandValue)(value));
}
exports.setOutput = setOutput;

Resources

@blazceltra blazceltra force-pushed the update-node-and-core-actions branch 2 times, most recently from f5c9dc7 to 704014e Compare February 10, 2025 06:27
Also bump github actions versions for checkout and setup-node.
@blazceltra blazceltra force-pushed the update-node-and-core-actions branch 3 times, most recently from 52cee1f to 332c975 Compare February 28, 2025 11:01
@blazceltra blazceltra force-pushed the update-node-and-core-actions branch 2 times, most recently from 003630b to a736e96 Compare February 28, 2025 11:03
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