Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

feat(utils): supportDeprecatedInfo forrule.meta.deprecated#10932

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Merged
Show file tree
Hide file tree
Changes fromall commits
Commits
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletionpackages/eslint-plugin/src/rules/no-empty-interface.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -16,7 +16,18 @@ export default createRule<Options, MessageIds>({
name: 'no-empty-interface',
meta: {
type: 'suggestion',
deprecated: true,
deprecated: {
deprecatedSince: '8.0.0',
replacedBy: [
{
rule: {
name: '@typescript-eslint/no-empty-object-type',
url: 'https://typescript-eslint.io/rules/no-empty-object-type',
},
},
],
url: 'https://github.com/typescript-eslint/typescript-eslint/pull/8977',
},
docs: {
description: 'Disallow the declaration of empty interfaces',
},
Expand Down
13 changes: 12 additions & 1 deletionpackages/eslint-plugin/src/rules/no-loss-of-precision.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -20,7 +20,18 @@ export default createRule<Options, MessageIds>({
meta: {
type: 'problem',
// defaultOptions, -- base rule does not use defaultOptions
deprecated: true,
deprecated: {
deprecatedSince: '8.0.0',
replacedBy: [
{
rule: {
name: 'no-loss-of-precision',
url: 'https://eslint.org/docs/latest/rules/no-loss-of-precision',
},
},
],
url: 'https://github.com/typescript-eslint/typescript-eslint/pull/8832',
},
docs: {
description: 'Disallow literal numbers that lose precision',
extendsBaseRule: true,
Expand Down
13 changes: 12 additions & 1 deletionpackages/eslint-plugin/src/rules/no-type-alias.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -37,7 +37,18 @@ export default createRule<Options, MessageIds>({
name: 'no-type-alias',
meta: {
type: 'suggestion',
deprecated: true,
deprecated: {
deprecatedSince: '6.0.0',
replacedBy: [
{
rule: {
name: '@typescript-eslint/consistent-type-definitions',
url: 'https://typescript-eslint.io/rules/consistent-type-definitions',
},
},
],
url: 'https://github.com/typescript-eslint/typescript-eslint/pull/6229',
},
docs: {
description: 'Disallow type aliases',
// too opinionated to be recommended
Expand Down
13 changes: 12 additions & 1 deletionpackages/eslint-plugin/src/rules/no-var-requires.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -15,7 +15,18 @@ export default createRule<Options, MessageIds>({
name: 'no-var-requires',
meta: {
type: 'problem',
deprecated: true,
deprecated: {
deprecatedSince: '8.0.0',
replacedBy: [
{
rule: {
name: '@typescript-eslint/no-require-imports',
url: 'https://typescript-eslint.io/rules/no-require-imports',
},
},
],
url: 'https://github.com/typescript-eslint/typescript-eslint/pull/8334',
},
docs: {
description: 'Disallow `require` statements except in import statements',
},
Expand Down
13 changes: 12 additions & 1 deletionpackages/eslint-plugin/src/rules/prefer-ts-expect-error.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -11,7 +11,18 @@ export default createRule<[], MessageIds>({
name: 'prefer-ts-expect-error',
meta: {
type: 'problem',
deprecated: true,
deprecated: {
deprecatedSince: '7.11.0',
replacedBy: [
{
rule: {
name: '@typescript-eslint/ban-ts-comment',
url: 'https://typescript-eslint.io/rules/ban-ts-comment',
},
},
],
url: 'https://github.com/typescript-eslint/typescript-eslint/pull/9081',
},
docs: {
description: 'Enforce using `@ts-expect-error` over `@ts-ignore`',
},
Expand Down
27 changes: 26 additions & 1 deletionpackages/eslint-plugin/src/rules/sort-type-constituents.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -121,7 +121,32 @@ export default createRule<Options, MessageIds>({
name: 'sort-type-constituents',
meta: {
type: 'suggestion',
deprecated: true,
deprecated: {
deprecatedSince: '7.13.0',
replacedBy: [
{
plugin: {
name: 'eslint-plugin-perfectionist',
url: 'https://perfectionist.dev',
},
rule: {
name: 'perfectionist/sort-intersection-types',
url: 'https://perfectionist.dev/rules/sort-intersection-types',
},
},
{
plugin: {
name: 'eslint-plugin-perfectionist',
url: 'https://perfectionist.dev',
},
rule: {
name: 'perfectionist/sort-union-types',
url: 'https://perfectionist.dev/rules/sort-union-types',
},
},
],
url: 'https://github.com/typescript-eslint/typescript-eslint/pull/9253',
},
docs: {
description:
'Enforce constituents of a type union/intersection to be sorted alphabetically',
Expand Down
62 changes: 61 additions & 1 deletionpackages/utils/src/ts-eslint/Rule.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -27,6 +27,64 @@ export interface RuleMetaDataDocs {
url?: string;
}

export interface ExternalSpecifier {
/**
* Name of the referenced plugin / rule.
*/
name?: string;
/**
* URL pointing to documentation for the plugin / rule.
*/
url?: string;
}

export interface ReplacedByInfo {
/**
* General message presented to the user, e.g. how to replace the rule
*/
message?: string;
/**
* URL to more information about this replacement in general
*/
url?: string;
/**
* Name should be "eslint" if the replacement is an ESLint core rule. Omit
* the property if the replacement is in the same plugin.
*/
plugin?: ExternalSpecifier;
/**
* Name and documentation of the replacement rule
*/
rule?: ExternalSpecifier;
}

export interface DeprecatedInfo {
/**
* General message presented to the user, e.g. for the key rule why the rule
* is deprecated or for info how to replace the rule.
*/
message?: string;
/**
* URL to more information about this deprecation in general.
*/
url?: string;
/**
* An empty array explicitly states that there is no replacement.
*/
replacedBy?: ReplacedByInfo[];
/**
* The package version since when the rule is deprecated (should use full
* semver without a leading "v").
*/
deprecatedSince?: string;
/**
* The estimated version when the rule is removed (probably the next major
* version). null means the rule is "frozen" (will be available but will not
* be changed).
*/
availableUntil?: string | null;
}

export interface RuleMetaData<
MessageIds extends string,
PluginDocs = unknown,
Expand All@@ -35,7 +93,7 @@ export interface RuleMetaData<
/**
* True if the rule is deprecated, false otherwise
*/
deprecated?: boolean;
deprecated?: boolean | DeprecatedInfo;
/**
* Documentation for the rule
*/
Expand All@@ -56,6 +114,8 @@ export interface RuleMetaData<
messages: Record<MessageIds, string>;
/**
* The name of the rule this rule was replaced by, if it was deprecated.
*
* @deprecated since eslint 9.21.0, in favor of `RuleMetaData#deprecated.replacedBy`
*/
replacedBy?: readonly string[];
/**
Expand Down
1 change: 0 additions & 1 deletionpackages/website/rulesMeta.ts
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -6,7 +6,6 @@ export const rulesMeta = Object.entries(rules).map(([name, content]) => ({
fixable: content.meta.fixable,
hasSuggestions: content.meta.hasSuggestions,
name,
replacedBy: content.meta.replacedBy,
type: content.meta.type,
}));

Expand Down
[8]ページ先頭
©2009-2025 Movatter.jp