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

[Breaking] Enforce minimum Node JS to v18 #4897

Closed

Conversation

Pranav-yadav
Copy link
Contributor

@Pranav-yadav Pranav-yadav commented Jun 29, 2023

Summary:

Enforce minimum Node JS version to v18 via /package.json#engines

Bumps in CI via:

  1. [Actions] CI - Bump Node JS to v18 in CI Pipeline #4898

Changelog:

[General] [Breaking] - Enforce minimum Node JS v18

Test Plan:

  • Should pass tests & builds successfully

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Jun 29, 2023
@Pranav-yadav
Copy link
Contributor Author

Note
This PR should be merged after #4898 to verify that the tests and builds pass successfully.

@passy
Copy link
Member

passy commented Jun 29, 2023

There are a few CI issues with our sub-projects. Could you take a look?

@passy
Copy link
Member

passy commented Jun 29, 2023

Ohh, sorry. This is a stack!

facebook-github-bot pushed a commit that referenced this pull request Jun 30, 2023
Summary:
This diff bumps Node JS to `v18` in CI Pipeline.
Related PR:
1. #4897

## Changelog

[General] [Changed] - Bump Node JS to `v18` in CI Pipeline

Pull Request resolved: #4898

Test Plan: - Should pass tests & builds successfully

Reviewed By: lblasa

Differential Revision: D47128828

Pulled By: passy

fbshipit-source-id: b8d829b273898bf5e297560fbbdbafba80285b33
@Pranav-yadav
Copy link
Contributor Author

Ohh, sorry. This is a stack!

Np.
Rebased just now, so CI checks should pass now.

@Pranav-yadav
Copy link
Contributor Author

Passing w/ Node 18.x 🎉:

  • ✅ Desktop Node CI / build (18.x, ubuntu-latest) (pull_request) Successful in 14m Details
  • ✅ Desktop Node CI / build (18.x, windows-latest) (pull_request) Successful in 20m Details
  • ✅ Desktop Node CI / build (18.x, macos-latest) (pull_request) Successful in 23m Details

@facebook-github-bot
Copy link
Contributor

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@passy
Copy link
Member

passy commented Jul 7, 2023

Woohoo! Let's ship it.

@facebook-github-bot
Copy link
Contributor

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@Pranav-yadav
Copy link
Contributor Author

Thanks for review ❤️.

@facebook-github-bot
Copy link
Contributor

@passy merged this pull request in 78d56ea.

@Pranav-yadav Pranav-yadav deleted the Pranav-yadav/bump-node18 branch July 21, 2023 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants