chore: enable no-export-all on index files

[tido64]

Some bundlers, such as esbuild, are not able to tree-shake nested export * statements. Fixed them using the no-export-all ESLint rule.

Platforms Impacted

• iOS
• macOS
• win32 (Office)
• windows
• android

Description of changes

Some bundlers, such as esbuild, are not able to tree-shake nested export * statements.

The Fluent team has also recently turned this on: microsoft/fluentui#22073

Fixed using the no-export-all ESLint rule.

Verification

Functionally, nothing has changed.

[rurikoaraki]

Out of curiosity, some index files are skipped, is that because they're not top-level files?

[tido64]

Out of curiosity, some index files are skipped, is that because they're not top-level files?

This is the rule I added:

fluentui-react-native/packages/framework/eslint-config-rules/eslintrc.js

Lines 19 to 22 in 0fd96ba

	files: '**/src/index.{js,ts,tsx}',
	rules: {
	'@rnx-kit/no-export-all': ['error', { expand: 'all' }],
	},

It should only catch the main file. Index files that are internal in the package are skipped for now.