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

Update react-native cli test fixture versions #1710

Merged
merged 7 commits into from
Apr 1, 2022

Conversation

Cawllec
Copy link
Contributor

@Cawllec Cawllec commented Mar 30, 2022

Goal

Update the tested versions of react-native in the react-native-cli tests to ensure we have coverage up to the newest release of react-native. 0.60 is still our baseline and each version between that and the latest (0.67) will have tests run against them, including an ejected Expo app and an app running on the Hermes engine.

Since there are a large amount of file changes, I would advise reviewing each commit individually.

@github-actions
Copy link

github-actions bot commented Mar 30, 2022

@bugsnag/browser bundle size diff

Minified Minfied + Gzipped
Before 42.31 kB 12.94 kB
After 42.31 kB 12.94 kB
± No change No change

code coverage diff

Coverage values did not change👌.

Total:

Lines Branches Functions Statements
0%(+0%) 0%(+0%) 0%(+0%) 0%(+0%)

Generated by 🚫 dangerJS against db6620d

@imjoehaines
Copy link
Contributor

Looks like there's an issue with the 0.66 fixture, it's consistently failing here: https://buildkite.com/bugsnag/bugsnag-js-react-native-cli/builds/2069#366823ef-addc-4e89-afa2-25155166d636/378-396

@Cawllec
Copy link
Contributor Author

Cawllec commented Apr 1, 2022

Looks like there's an issue with the 0.66 fixture, it's consistently failing here: https://buildkite.com/bugsnag/bugsnag-js-react-native-cli/builds/2069#366823ef-addc-4e89-afa2-25155166d636/378-396

Yeah, I'd typo'd a reference in the docker file, that's fixed now

@Cawllec Cawllec merged commit 59e4de9 into next Apr 1, 2022
@Cawllec Cawllec deleted the testing/rn-cli-version-update branch April 1, 2022 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants