Skip to content
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

Bump action Node version to v20 #148

Merged
merged 8 commits into from
Mar 27, 2024

Conversation

LukasTy
Copy link
Contributor

@LukasTy LukasTy commented Feb 19, 2024

Node 16 is deprecated on GitHub actions.
Running this action produces the following warning:
https://github.com/mui/mui-x/actions/runs/7956989267

Changes

  • Update using to node20 instead of node16 (other actions treated this change as a major release)
  • Bump @types/node to v20 Already done via a separate PR
  • Update @actions/core and @actions/io to the latest versions Already done via a separate PR
  • Replace @octokit/webhooks-definitions with @octokit/webhooks-types as the former is already deprecated
  • Run format

Question

Do I need to run build and commit the updated built file or is release handled separately? 🤔

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: bug, enhancement, maintenance

@LukasTy LukasTy changed the title Use not-deprecated package Bump action Node version to v20 Feb 19, 2024
@carloscastrojumo
Copy link
Owner

Hey @LukasTy . Thank you for this. Sorry for the late reply. Do you mind rebase your PR and fix conflicts?

And yes, there is no need to commit the output of the build command.

@LukasTy
Copy link
Contributor Author

LukasTy commented Mar 27, 2024

Hey @carloscastrojumo
I've updated the PR and bumped the changed package to latest version.

Copy link
Owner

@carloscastrojumo carloscastrojumo left a comment

Choose a reason for hiding this comment

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

Thanks @LukasTy . will release this.

@carloscastrojumo carloscastrojumo merged commit e5d2287 into carloscastrojumo:main Mar 27, 2024
3 checks passed
@LukasTy LukasTy deleted the update-node-to-20 branch March 28, 2024 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants