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

chore(cli, metro): bump cli and metro to latest ahead of 71 #35107

Closed
wants to merge 2 commits into from

Conversation

kelset
Copy link
Contributor

@kelset kelset commented Oct 27, 2022

Summary

Small PR with bump to the new versions of CLI and Metro in preparation of the branch cut for 0.71.

While at it, did a cheeky npx yarn-deduplicate to clean up a bit the deps.

Changelog

[General] [Changed] - Bump CLI to 10.0.0-alpha.1 and Metro to 0.73.3

Test Plan

CI is green

@kelset kelset requested a review from hramos as a code owner October 27, 2022 15:31
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. p: Microsoft Partner: Microsoft Partner labels Oct 27, 2022
@github-actions
Copy link

Warnings
⚠️ 🔒 package.json - Changes were made to package.json. This will require a manual import by a Facebook employee.

Generated by 🚫 dangerJS against 4c1f750

@facebook-github-bot
Copy link
Contributor

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

@motiz88
Copy link
Contributor

motiz88 commented Oct 27, 2022

For the record, these are the changelogs for this Metro update: 0.73.0, 0.73.1, 0.73.2, 0.73.3

Ideally we should mention relevant items from the Metro changelog in the RN 71 changelog - might be worth thinking about a process / format for these PRs to help the person who will compile the RN changelog. Here in particular I'd call out:

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: 475310d
Branch: main

@kelset
Copy link
Contributor Author

kelset commented Oct 27, 2022

Great stuff, thanks @motiz88! and yeah, totally agree (and we also need to come up with something similar for Hermes 😅)

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @kelset in f550606.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Oct 27, 2022
@kelset kelset deleted the kelset/bump-cli-and-metro-for-71 branch October 28, 2022 08:00
OlimpiaZurek pushed a commit to OlimpiaZurek/react-native that referenced this pull request May 22, 2023
…#35107)

Summary:
Small PR with bump to the new versions of CLI and Metro in preparation of the branch cut for 0.71.

While at it, did a cheeky `npx yarn-deduplicate` to clean up a bit the deps.

## Changelog

<!-- Help reviewers and the release process by writing your own changelog entry. For an example, see:
https://reactnative.dev/contributing/changelogs-in-pull-requests
-->

[General] [Changed] - Bump CLI to 10.0.0-alpha.1 and Metro to 0.73.3

Pull Request resolved: facebook#35107

Test Plan: CI is green

Reviewed By: motiz88

Differential Revision: D40762683

Pulled By: huntie

fbshipit-source-id: e523a49c78588ca80351f44cb02bcd4c0137475e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Contributor A React Native contributor. Merged This PR has been merged. p: Microsoft Partner: Microsoft Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants