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
Copy link
Copy Markdown

@simao-silva simao-silva commented May 11, 2026

ENG-1363

Changes:

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

@simao-silva simao-silva force-pushed the ci/eng-1363/fix-node24-deprecation-warning branch from 1b8f6a7 to 49358c3 Compare May 11, 2026 02:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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
File Description
README.md Markdown whitespace cleanup in usage/docs sections.
action.yaml Pin actions/setup-python to a specific commit SHA and remove extra whitespace in the composite step script.
.releaserc.yaml Remove trailing whitespace in assets configuration.
.github/workflows/release.yaml Pin action versions (by SHA) and add minor formatting whitespace/newlines.
.github/workflows/checks.branches.yaml Whitespace/formatting change to branch ignore config (currently introduces invalid YAML).
.github/workflows/checks.base.yaml Pin checkout + commitlint action versions and update commitlint config path to .mjs.
.github/configs/commitlint.config.mjs Add ESM-based commitlint configuration.
.github/configs/commitlint.config.js Remove 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
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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-aveiro May 11, 2026 23:30
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