[android] Remove a check that is no longer needed (#535)

chrfalch · web-flow · commit 8f99d4dbe286 · 2024-10-16T12:46:57.000-04:00
* Removed a check that is no longer needed We've had some issue with monorepos when resolving the codegen config on Android in other libraries, and found that due to versions no longer in use we could now remove the version test in `react-native-config.js` and always return the component descriptor etc. (The monorepo issue was with using `require.main.require` which will resolve to the calling script and not the module) For reference, here is the same PR in @react-native-community/slider: callstack/react-native-slider#657 * cr: Reverted and simplified change after code review Removed previous change and made the `supportsCodegenConfig` variable true initially * revert: reverted commit - removed last line.
diff --git a/react-native.config.js b/react-native.config.js
@@ -1,4 +1,4 @@
-let supportsCodegenConfig = false;
+let supportsCodegenConfig = true;
 try {
   const rnCliAndroidVersion = require.main.require(
     '@react-native-community/cli-platform-android/package.json',

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-let supportsCodegenConfig = false;`
	`1`	`+let supportsCodegenConfig = true;`
`2`	`2`	`try {`
`3`	`3`	`const rnCliAndroidVersion = require.main.require(`
`4`	`4`	`'@react-native-community/cli-platform-android/package.json',`