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

fix(0.71-stable/tests): attempting to fix iOS template tests #35963

Merged
merged 3 commits into from
Jan 26, 2023

Conversation

hoxyq
Copy link
Contributor

@hoxyq hoxyq commented Jan 25, 2023

Currently receiving weird errors on CircleCI machines for iOS, android ones work well
https://app.circleci.com/pipelines/github/facebook/react-native/19037/workflows/7987fa61-03d1-4200-910b-ebca0ec04054

Moved echo "//localhost:4873/:_authToken=secretToken" > .npmrc before bootstrapping verdaccio, looks like there was a race-condition on node 16+

Still don't know why this works well on main branch

@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. p: Facebook Partner: Facebook Partner labels Jan 25, 2023
@hoxyq hoxyq force-pushed the 0.71-stable-fix-verdaccio-on-ios branch 3 times, most recently from 8291ed1 to 8226ef6 Compare January 25, 2023 14:41
Summary:
Removing a stale configuration that was configuring username/password before publishing to NPM. This is effectively unused + the Github Token there is invalid therefore can be removed.

## Changelog

[INTERNAL] - Remove unused .netrc file from CircleCI

Pull Request resolved: facebook#35785

Test Plan: n/a

Reviewed By: cipolleschi

Differential Revision: D42385163

Pulled By: cortinico

fbshipit-source-id: 0dbbf44459d59f792da4221d6100800a2f4efda2
@hoxyq hoxyq force-pushed the 0.71-stable-fix-verdaccio-on-ios branch 10 times, most recently from f11ba03 to 55bba8f Compare January 25, 2023 20:21
Summary:
Node 18: https://nodejs.org/de/blog/announcements/v18-release-announce/

Node 16 EOL: 2023-09-11
https://nodejs.org/en/blog/announcements/nodejs16-eol/

Node 18 EOL: 2025-04-30

Follow-up
- facebook#34171
- facebook#32980

Ref
- react-native-community/docker-android#187

cc ramonmedel cortinico gengjiawen

## 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
-->

[JavaScript] [Changed] - Bump node version from 16 to 18

Pull Request resolved: facebook#35443

Test Plan: Everything builds and runs as expected

Reviewed By: cipolleschi

Differential Revision: D41531161

Pulled By: cortinico

fbshipit-source-id: 305888f55ed179f75bef34548aebf22fc2951308
@hoxyq hoxyq force-pushed the 0.71-stable-fix-verdaccio-on-ios branch from 55bba8f to 9f82b9c Compare January 25, 2023 20:34
Copy link
Contributor

@kelset kelset left a comment

Choose a reason for hiding this comment

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

great stuff! Let's put it in

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. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants