resetCache and others from metro.config.json get overwritten

[jens-duttke]

• Review the documentation: https://facebook.github.io/react-native
• Search for existing issues: https://github.com/facebook/react-native/issues
• Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment

React Native Environment Info:
System:
OS: Windows 10
CPU: x64 Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz
Memory: 5.73 GB / 15.88 GB
Binaries:
npm: 6.1.0 - C:\Program Files\nodejs\npm.CMD
IDEs:
Android Studio: Version 3.2.0.0 AI-181.5540.7.32.5014246

Description

Regarding https://facebook.github.io/metro/docs/en/configuration.html, it should be possible to set resetCache in the metro.config.json. Unfortunately, in react-native/local-cli/server/runServer.js line 64, the config option is overwritten by the command line arguments, without to consider that it's already set in the metro.config.json.
The same count for other options like projectRoot, watchFolders, server.port, maxWorkers ...

Reproducible Demo

In any project, create a metro.config.json with this content:

{
	"resetCache": true
}

It will not clear the cache.

[Salakar]

Hey @jens-duttke - thanks so much for reporting this with a reproducible example and pinpointing that the issue lies with the option being overridden by the CLI.

We'll move this issue to the new CLI community repo for review.

[cpojer]

This issue has been moved to react-native-community/cli#249.