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.

Sign up for GitHub

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

Jump to bottom

React Native Version Matrix CI #2673

Merged
merged 43 commits into from
Dec 19, 2022
Merged

React Native Version Matrix CI #2673

merged 43 commits into from
Dec 19, 2022

Conversation

krystofwoldrich
Copy link
Member

@krystofwoldrich krystofwoldrich commented Dec 5, 2022 •
edited
Loading

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

@krystofwoldrich krystofwoldrich changed the title Add clean 0.64.4 test app React Native Version Matrix CI Dec 5, 2022
@krystofwoldrich
Remove placeholder brackets from if condition of the build steps
0a78cd6
@krystofwoldrich
Remove placeholder brackets from the end of conditions
1a7c869
@krystofwoldrich
Add bracket to enclose legacy arch condition
56ba1aa
@krystofwoldrich
Add missing bracket to close build conditions
cfba6ce
@krystofwoldrich
Add pod install
e89d844
@krystofwoldrich
Add cancel previous workflow
bb02928
@krystofwoldrich
Fixed derivedData dir
80cffb5
@krystofwoldrich
Add rn-init patch, add sdk to the sample ci jobs
d183c82
@krystofwoldrich
Fix add yalc
6f82109
@krystofwoldrich
Merge remote-tracking branch 'origin/main' into rn-matrix-e2e-tests
3c8dc60
@krystofwoldrich
Remove silent from patch
7d5283b
@krystofwoldrich
Delete plain test app from the repo
7e04ea0
@krystofwoldrich
Use rn diff repo af source of the plain rn app
808900c
@krystofwoldrich
Use https for the rn diff repo
eca6c1b
@krystofwoldrich
Fix the test app cwd path
37fa1e2
@krystofwoldrich
Fix patch relative path
c50bc76
@krystofwoldrich
Add more rn versions
3c10900
@krystofwoldrich
Revert "Add more rn versions"
e6afe40 
This reverts commit 3c10900.
@krystofwoldrich
Fix patch job name typo
2f034d4
@krystofwoldrich
Make patch tool verbose and allow larger diff
25c9f22
@krystofwoldrich
WIP! xcode patch
59d73a5
@krystofwoldrich
Finish xcode patch
a859ec2
@krystofwoldrich
Fix the xcode project name
dfe20a6
@krystofwoldrich
Add build type and new arch compatible rn version
9877670
@krystofwoldrich krystofwoldrich mentioned this pull request Dec 9, 2022
Merged
@krystofwoldrich krystofwoldrich marked this pull request as ready for review December 9, 2022 13:59
@krystofwoldrich
Copy link
Member Author

@marandaneto Maybe we could run these version builds only on the main branch, to save a bit of CI minutes?

@krystofwoldrich krystofwoldrich merged commit d8bb1e6 into main Dec 19, 2022
@krystofwoldrich krystofwoldrich deleted the rn-matrix-e2e-tests branch December 19, 2022 13:29
@krystofwoldrich krystofwoldrich mentioned this pull request Jan 10, 2023
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marandaneto marandaneto marandaneto approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@krystofwoldrich @marandaneto