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

Downgrade CLI to alpha.0 to get Metro working again in CI #36654

Merged
merged 1 commit into from Mar 27, 2023

Conversation

cipolleschi
Copy link
Contributor

Summary

This PR is part of the work @huntie is doing to fix the CLI-Metro situation. For the time being, this local commit should bring back CircleCI to a green state.

Changelog

[General][Fixed] - Downgrade CLI to alpha.0 to make Metro work again

Test Plan

  • CircleCI must be green

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 27, 2023
@cipolleschi cipolleschi changed the base branch from main to 0.72-stable March 27, 2023 14:37
@robhogan
Copy link
Contributor

(Not just CI, but this is necessary to unbreak RC.)

@cipolleschi cipolleschi merged commit 17a9615 into 0.72-stable Mar 27, 2023
3 checks passed
integrity sha512-IdZj/2CZYME/iWY4cAxKBWAr/pQTfj5I2NZp8Z4+u6/KrjP2PuoMkUcb+ONvwQCLpd9R62EnuaPzXVomH43d8A==
"@react-native-community/cli-clean@^11.0.0-alpha.0":
version "11.0.0"
resolved "http://localhost:4873/@react-native-community%2fcli-clean/-/cli-clean-11.0.0.tgz#7225f8df011893de1cb740a0cad3dd2670574da5"
Copy link
Contributor

Choose a reason for hiding this comment

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

@cipolleschi - sorry, just spotted these localhost URLs in the lockfile.

That might be OK for CI if that's the Verdaccio host, but won't work for contributors. Not sure if this is related to alpha-0 being unpublished?

Copy link
Contributor

Choose a reason for hiding this comment

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

Working on a follow-up PR to fix these now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

My bad, actually I though I pushed a change to use overrides as @huntie did here: 155591b. I don't know why those changes have not been kept. :/

Copy link
Contributor

Choose a reason for hiding this comment

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

Working on a follow-up PR to fix these now.

#36661

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner Pick Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants