Skip to content

Commit

Permalink
Fix default projectRoot + watchFolders computation
Browse files Browse the repository at this point in the history
Summary:
@public

This diff fixes the `projectRoot` calculation on React Native when the app does not have a config file. This should fix the issues reported in #20712

Reviewed By: hramos

Differential Revision: D9444982

fbshipit-source-id: 4cb41fa5224d2addf92976cc119e49dea6a6346b
  • Loading branch information
rafeca authored and facebook-github-bot committed Aug 23, 2018
1 parent b620cca commit 9a77ff5
Showing 1 changed file with 7 additions and 9 deletions.
16 changes: 7 additions & 9 deletions local-cli/util/Config.js
Expand Up @@ -21,7 +21,7 @@ const {loadConfig} = require('metro-config');
*/
import type {ConfigT} from 'metro-config/src/configTypes.flow';

function getProjectPath() {
function getProjectRoot() {
if (
__dirname.match(/node_modules[\/\\]react-native[\/\\]local-cli[\/\\]util$/)
) {
Expand All @@ -44,12 +44,12 @@ const resolveSymlinksForRoots = roots =>
[...roots],
);

const getProjectRoots = () => {
const getWatchFolders = () => {
const root = process.env.REACT_NATIVE_APP_ROOT;
if (root) {
return resolveSymlinksForRoots([path.resolve(root)]);
}
return resolveSymlinksForRoots([getProjectPath()]);
return [];
};

const getBlacklistRE = () => {
Expand All @@ -76,17 +76,15 @@ const Config = {
],
getPolyfills,
},

watchFolders: [getProjectPath(), ...getProjectRoots()],
watchFolders: getWatchFolders(),
transformModulePath: require.resolve('metro/src/reactNativeTransformer'),
},

getProjectPath,
getProjectRoots,

async load(configFile: ?string): Promise<ConfigT> {
const argv = {cwd: getProjectRoot()};

return await loadConfig(
configFile ? {config: configFile} : {},
configFile ? {...argv, config: configFile} : argv,
this.DEFAULT,
);
},
Expand Down

0 comments on commit 9a77ff5

Please sign in to comment.