-
Notifications
You must be signed in to change notification settings - Fork 24.3k
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
[RN][OSS][iOS] Add Matrix tests for Flipper/NoFlipper #34595
Conversation
63e23a7
to
a6edc8e
Compare
805c8d9
to
d6075b6
Compare
d6075b6
to
cd16be4
Compare
Base commit: 5c109b3 |
Base commit: 5c109b3 |
@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love seeing this stuff. This looks good to me
What follows is no criticism it's just excitement for future - now just hoping for macCatalyst and use_frameworks :-)
🚂
Yup, baby steps! :D |
This pull request was successfully merged by @cipolleschi in bf6a24b. When will my fix make it into a release? | Upcoming Releases |
Summary: This PR adds some tests to verify that we can build the template in every Debug configuration using Flipper and without Flipper. ## Changelog [iOS] [Added] - Add CircleCI tests to verify that we can run the Template with and without Flipper Pull Request resolved: facebook#34595 Test Plan: CircleCI is green Reviewed By: cortinico Differential Revision: D39262137 Pulled By: cipolleschi fbshipit-source-id: dae45b106cd13fb69442ea216005cee114d861f4
Summary
This PR adds some tests to verify that we can build the template in every Debug configuration using Flipper and without Flipper.
Changelog
[iOS] [Added] - Add CircleCI tests to verify that we can run the Template with and without Flipper
Test Plan
CircleCI is green