-
Notifications
You must be signed in to change notification settings - Fork 2.8k
chore: migrate v9 packages to use eslint flat configs #35418
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.
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
base: master
Are you sure you want to change the base?
chore: migrate v9 packages to use eslint flat configs #35418
Conversation
eca3be2 to
7d548f5
Compare
📊 Bundle size reportUnchanged fixtures
|
4716d92 to
5897acd
Compare
|
Pull request demo site: URL |
change/@fluentui-babel-preset-global-context-705679a5-80b4-48f7-bb30-1a11814f14da.json
Show resolved
Hide resolved
5897acd to
7d55897
Compare
| "@fluentui/scripts-storybook": "*", | ||
| "@fluentui/eslint-plugin": "*" | ||
| }, | ||
| "scripts": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is temporary change, will be removed once we will use default target without flag to support legacy format
| module.exports = tseslint.config( | ||
| ...core, | ||
| /** @type { import("eslint").Linter.Config } */ | ||
| module.exports = defineConfig( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can use now default defineConfig from eslint after we upgraded ts-eslint in #35378
57e4027 to
0971f0d
Compare

This pull request updates multiple Fluent UI packages by migrating their linting configuration to use the new ESLint flat config format
partially implements #32289