Skip to content

Commit abf466d

Browse files
JoshuaKGoldbergbrettz9
authored andcommitted
Remove default text check
1 parent ede2494 commit abf466d

File tree

2 files changed

+4
-15
lines changed

2 files changed

+4
-15
lines changed

src/rules/checkTagNames.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ export default iterateJsdoc(({
114114
return false;
115115
}
116116

117-
if (jsdocTag.description.trim() === 'default' || jsdocTag.name === 'default') {
117+
if (jsdocTag.name === 'default') {
118118
return false;
119119
}
120120

@@ -155,7 +155,7 @@ export default iterateJsdoc(({
155155
}
156156

157157
if (tagIsRedundantWhenTyped(jsdocTag)) {
158-
reportWithTypeRemovalFixer(`'@${jsdocTag.tag}' is generally redundant outside of \`declare\` contexts when using a type system.`, jsdocTag, tagIndex);
158+
reportWithTypeRemovalFixer(`'@${jsdocTag.tag}' is redundant outside of ambient (\`declare\`/\`.d.ts\`) contexts when using a type system.`, jsdocTag, tagIndex);
159159
return true;
160160
}
161161

test/rules/assertions/checkTagNames.js

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ export default {
151151
errors: [
152152
{
153153
line: 2,
154-
message: '\'@abstract\' is generally redundant outside of `declare` contexts when using a type system.',
154+
message: '\'@abstract\' is redundant outside of ambient (`declare`/`.d.ts`) contexts when using a type system.',
155155
},
156156
],
157157
options: [
@@ -173,7 +173,7 @@ export default {
173173
errors: [
174174
{
175175
line: 3,
176-
message: '\'@abstract\' is generally redundant outside of `declare` contexts when using a type system.',
176+
message: '\'@abstract\' is redundant outside of ambient (`declare`/`.d.ts`) contexts when using a type system.',
177177
},
178178
],
179179
options: [
@@ -938,17 +938,6 @@ export default {
938938
],
939939
parser: require.resolve('@typescript-eslint/parser'),
940940
},
941-
{
942-
code: `
943-
/** @abstract - default */
944-
let a;
945-
`,
946-
options: [
947-
{
948-
typed: true,
949-
},
950-
],
951-
},
952941
{
953942
code: `
954943
/** @template name */

0 commit comments

Comments
 (0)