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

Fix regression in dynamic resolution for out-of-tree platforms #20662

Closed
wants to merge 15 commits into from
Closed

Fix regression in dynamic resolution for out-of-tree platforms #20662

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
d314ef8
Initial changes to make platform resolution dynamic for out-of-tree p…
matthargett Aug 14, 2018
94425cc
Merge branch 'master' of github.com:matthargett/react-native into dyn…
matthargett Aug 15, 2018
a5190b7
Revert fixes for lint/prettier failures.
matthargett Aug 16, 2018
a51bffe
Merge branch 'master' of github.com:facebook/react-native into dynami…
matthargett Aug 16, 2018
771df00
Take Name.es6.js and Name.es7.js into account for regexp.
matthargett Aug 16, 2018
403930c
Revert prettier auto-fix.
matthargett Aug 16, 2018
1f7f362
Adjust comment and regexp. Note weird error when only two platforms a…
matthargett Aug 17, 2018
8dc4fe7
From @rozele: React Native overrides the settings for, e.g., , which…
matthargett Aug 22, 2018
9edfa98
Merge branch 'master' of github.com:facebook/react-native into dynami…
matthargett Aug 22, 2018
5207cd1
Remove hard-coded platform paths.
matthargett Aug 22, 2018
0e72049
hasteImpl will be overridden via metro configurations of out-of-tree …
matthargett Aug 23, 2018
3159e0c
Merge branch 'master' of github.com:facebook/react-native into dynami…
matthargett Aug 23, 2018
d366ea4
Update tests to remove out-of-tree platform specifics
matthargett Aug 23, 2018
9ce4e7a
Fix comment
matthargett Aug 23, 2018
5733fe1
My synthetic tests didn't capture real-world pathing. This should fix…
matthargett Aug 23, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Revert prettier auto-fix.
  • Loading branch information
@matthargett
matthargett committed Aug 16, 2018
commit 403930c9f293ce48ef853d4ecbb55c2724bf8fa8
6 changes: 1 addition & 5 deletions local-cli/generator/templates.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -171,11 +171,7 @@ function createFromRemoteTemplate(
// only for publishing the template to npm.
// We want to ignore this dummy file, otherwise it would overwrite
// our project's package.json file.
ignorePaths: [
'package.json',
'dependencies.json',
'devDependencies.json',
],
ignorePaths: ['package.json', 'dependencies.json','devDependencies.json'],
});
installTemplateDependencies(templatePath, yarnVersion);
installTemplateDevDependencies(templatePath, yarnVersion);
Expand Down