-
Notifications
You must be signed in to change notification settings - Fork 50
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
Remove ART #89
Remove ART #89
Conversation
ART has been removed in React Native v0.62. See facebook/react-native@9a0ebe0 and https://github.com/react-native-community/art.
@root-devs @lelandrichardson Is there a chance this PR can be merged and get a new npm release soon? React Native 0.63 is out as well and this package is an important part of our suite of testing and we need to migrate to latest RN. I'd really appreciate if we can this out! 🙏 |
approve please |
@root-devs @lelandrichardson can you approve this ? 🙏 |
Hi @root-devs @lelandrichardson , we are working to migrate our react-native version to 0.63.2. The approval of this PR is crucial to our migration. Can you approve this, please? |
@root-devs @lelandrichardson What's stopping this from being merged? |
anyone? Approve? |
Can this change be merged to allow us to migration to react-native version 0.63.2? We currently have our tests disabled to allow our CI to pass. Not optimal but our tests are legacy and will require some effort to rewrite. |
@root-devs @kukiron |
I folked this fixed commit and published to my own npm registry. https://www.npmjs.com/package/dl-react-native-mock-render Why published to npm registry instead of installing from github? Because I met this issue yarnpkg/yarn#7212 at CI phrase of github action |
I tried using this but seems like the repo is not there anymore. Is there a different repo? |
ART has been removed in React Native v0.62. See facebook/react-native@9a0ebe0 and @react-native-community/art. Fixes #47.