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

feat(deps): use specified node version for build #1435

Merged
merged 2 commits into from
Nov 29, 2023
Merged

Conversation

Rotzbua
Copy link
Contributor

@Rotzbua Rotzbua commented Sep 24, 2023

What is the current behavior?

Node version for build environment is defined in the GH Actions scripts.

Issue Number

none

What is the new behavior?

Use central config file to manage node version.

Does this PR introduce a breaking change?

  • Yes
  • No

Specific Instructions

none

Other information

none

Copy link
Member

@Red-Asuka Red-Asuka left a comment

Choose a reason for hiding this comment

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

@Rotzbua node-version-file seems to be supported only in versions above actions/setup-node 2.5.0, please update actions/setup-node version.

https://github.com/actions/setup-node/releases/tag/v2.5.0

@ysfscream ysfscream added this to the v1.9.7 milestone Sep 25, 2023
@ysfscream
Copy link
Member

@Rotzbua
Thank you for your PR! You mentioned moving the Node version definition from the GH Actions scripts to using a centralized config file, which is a great improvement.

Before merging, I'd like to confirm one thing: Have you tested to ensure that we can still build a functional package after this change? We want to ensure this modification doesn't adversely affect our current build process.

I'm looking forward to hearing back from you!

@ysfscream ysfscream merged commit 342f756 into emqx:main Nov 29, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants