Skip to content

ci(ENG-1363): Update actions versions to resolve Node.js 20 deprecation warnings - #10

Open
simao-silva wants to merge 3 commits into
stagingfrom
ci/eng-1363/fix-node24-deprecation-warning
Open

ci(ENG-1363): Update actions versions to resolve Node.js 20 deprecation warnings#10
simao-silva wants to merge 3 commits into
stagingfrom
ci/eng-1363/fix-node24-deprecation-warning

Conversation

@simao-silva

@simao-silvasimao-silva commented May 11, 2026

Copy link
Copy Markdown

ENG-1363

Changes:

  • Bumped version of actions to resolve Node.js 20 deprecation warnings;
  • Removed unnecessary white spaces.

@simao-silva
simao-silvaforce-pushed the ci/eng-1363/fix-node24-deprecation-warning branch from 1b8f6a7 to 49358c3CompareMay 11, 2026 02:07

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates GitHub Actions dependencies/configuration to address Node.js 20-related deprecation warnings in CI, and applies whitespace/formatting cleanups across workflows and documentation.

Changes:

  • Pin/upgrade several GitHub Actions used in CI/release workflows (e.g., checkout/setup-node/setup-python, commitlint action).
  • Switch commitlint configuration from CommonJS (.js) to ESM (.mjs) and update the workflow to reference it.
  • Remove trailing whitespace / adjust minor formatting in YAML and README.

Reviewed changes

Copilot reviewed 5 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
FileDescription
README.mdMarkdown whitespace cleanup in usage/docs sections.
action.yamlPin actions/setup-python to a specific commit SHA and remove extra whitespace in the composite step script.
.releaserc.yamlRemove trailing whitespace in assets configuration.
.github/workflows/release.yamlPin action versions (by SHA) and add minor formatting whitespace/newlines.
.github/workflows/checks.branches.yamlWhitespace/formatting change to branch ignore config (currently introduces invalid YAML).
.github/workflows/checks.base.yamlPin checkout + commitlint action versions and update commitlint config path to .mjs.
.github/configs/commitlint.config.mjsAdd ESM-based commitlint configuration.
.github/configs/commitlint.config.jsRemove CommonJS-based commitlint configuration.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread.github/workflows/checks.branches.yaml Outdated

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 8 changed files in this pull request and generated no new comments.

@simao-silva
simao-silva marked this pull request as ready for review May 11, 2026 23:29
@simao-silva
simao-silva requested review from a team, jmartinhoj and joao-aveiroMay 11, 2026 23:30
Sign up for freeto 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

@simao-silva