Skip to content

Commit

Permalink
review feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
dmytrokirpa committed Dec 13, 2024
1 parent 2d8c8c8 commit 0386842
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 13 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,10 +57,6 @@ module.exports = {

This rule ensures the use of Fluent UI v9 counterparts for Fluent UI v8 components.

#### Options

- `unstable` (boolean): Whether to enforce Fluent UI v9 preview component migrations.

#### Examples

**✅ Do**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,7 @@ export const plugin: {
};
};
rules: {
"prefer-fluentui-v9": RuleModule<"replaceFluent8With9" | "replaceIconWithJsx" | "replaceStackWithFlex" | "replaceFocusZoneWithTabster", {
preview?: boolean | undefined;
}[], unknown, RuleListener>;
"prefer-fluentui-v9": RuleModule<"replaceFluent8With9" | "replaceIconWithJsx" | "replaceStackWithFlex" | "replaceFocusZoneWithTabster", {}[], unknown, RuleListener>;
};
};

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,12 +13,7 @@ export const rule = createRule<Options, MessageIds>({
docs: {
description: 'This rule ensures the use of Fluent UI v9 counterparts for Fluent UI v8 components.',
},
schema: [
{
type: 'object',
properties: {},
},
],
schema: [],
messages: {
replaceFluent8With9: `Avoid importing {{ fluent8 }} from '@fluentui/react', as this package has started migration to Fluent UI 9. Import {{ fluent9 }} from '{{ package }}' instead.`,
replaceIconWithJsx: `Avoid using Icon from '@fluentui/react', as this package has already migrated to Fluent UI 9. Use a JSX SVG icon from '@fluentui/react-icons' instead.`,
Expand Down

0 comments on commit 0386842

Please sign in to comment.