-
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
Fix symlink resolving for examples in the repo #12435
Conversation
A similar fix, among others, were posted 2 days ago, #12400. Aside from the addtional fixes, I like this cleaner approach to config btw, the hardcoded |
Hi @Swaagie! Thanks for your PR, it looks really great. I think we can find a way how we can combine our PRs and ship both. I love the work you did with scoped modules, that's just great! Also, you caught a bug with Maybe we can wait someone to review our PRs and find how we can merge them. I think in either case the merge is not going to be that big. |
Sounds like a plan. |
Thanks for the quick fix! |
3e6ba00
to
316f9ae
Compare
Rebased in a hope it'll fix CI builds |
I can not repro the issue on master. Do we still need this? |
By can't repro, do you mean it doesn't crawl to folders above, or that you just don't see the same error? My issue description is specific to my computer—if you don't have a broken symlink, you won't see the same error. But the problem is it crawls parent directories in the first place. |
I just double-checked the code in
|
I'd also like to explain the reason why this issue occurs: there is the line that defines a path to Let me know if you need any additional information to review this PR. Also, feel free to ping me on Slack. |
Thanks @Kureev, this is a well documented fix. My apologies for glossing over the important detail. I'll land it now. |
@ericvicenti has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Summary: Motivation: Few days ago gaearon [filed an issue](facebook/react-native#12406) that examples in the react-native repo doesn't work after the [recent changes in local-cli](facebook/react-native@bce6ece). This PR fixes reported bug. **Test plan (required)** - No UI changes - [x] Run UIExplorer from XCode (no package.json in the folder) and check if packager/application runs correctly cc davidaurelio satya164 grabbou Closes facebook/react-native#12435 Differential Revision: D4657370 Pulled By: ericvicenti fbshipit-source-id: 72ee4b96cae37c7ed2794ed4490ce7b4fbbd66c3
Motivation:
Few days ago @gaearon filed an issue that examples in the react-native repo doesn't work after the recent changes in local-cli. This PR fixes reported bug.
Test plan (required)
cc @davidaurelio @satya164 @grabbou