Skip to content

Commit 02c2b22

Browse files
authored
Merge pull request #244 from brettz9/meta.type
- Add `meta.type`
2 parents 832399f + 8b3d7ba commit 02c2b22

24 files changed

+91
-12
lines changed

src/rules/checkAlignment.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,4 +23,8 @@ export default iterateJsdoc(({
2323
break;
2424
}
2525
}
26+
}, {
27+
meta: {
28+
type: 'layout'
29+
}
2630
});

src/rules/checkExamples.js

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -188,9 +188,14 @@ export default iterateJsdoc(({
188188
);
189189
});
190190
});
191-
}, {returns: [
192-
'ArrowFunctionExpression',
193-
'ClassDeclaration',
194-
'FunctionDeclaration',
195-
'FunctionExpression'
196-
]});
191+
}, {
192+
meta: {
193+
type: 'suggestion'
194+
},
195+
returns: [
196+
'ArrowFunctionExpression',
197+
'ClassDeclaration',
198+
'FunctionDeclaration',
199+
'FunctionExpression'
200+
]
201+
});

src/rules/checkIndentation.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,8 @@ export default iterateJsdoc(({
1111
if (reg.test(text)) {
1212
report('There must be no indentation.');
1313
}
14+
}, {
15+
meta: {
16+
type: 'layout'
17+
}
1418
});

src/rules/checkParamNames.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,4 +100,8 @@ export default iterateJsdoc(({
100100
}
101101

102102
validateParameterNamesDeep(targetTagName, jsdocParameterNamesDeep, report);
103+
}, {
104+
meta: {
105+
type: 'suggestion'
106+
}
103107
});

src/rules/checkSyntax.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,4 +14,8 @@ export default iterateJsdoc(({
1414
break;
1515
}
1616
}
17+
}, {
18+
meta: {
19+
type: 'suggestion'
20+
}
1721
});

src/rules/checkTagNames.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ export default iterateJsdoc(({
2424
});
2525
}, {
2626
meta: {
27-
fixable: 'code'
27+
fixable: 'code',
28+
type: 'suggestion'
2829
}
2930
});

src/rules/checkTypes.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ export default iterateJsdoc(({
6161
});
6262
}, {
6363
meta: {
64-
fixable: 'code'
64+
fixable: 'code',
65+
type: 'suggestion'
6566
}
6667
});

src/rules/implementsOnClasses.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,4 +15,8 @@ export default iterateJsdoc(({
1515
}
1616

1717
report('@implements used on a non-constructor function');
18+
}, {
19+
meta: {
20+
type: 'suggestion'
21+
}
1822
});

src/rules/matchDescription.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,4 +49,8 @@ export default iterateJsdoc(({
4949

5050
return validateDescription(description, tag.tag);
5151
});
52+
}, {
53+
meta: {
54+
type: 'suggestion'
55+
}
5256
});

src/rules/newlineAfterDescription.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ export default iterateJsdoc(({
5555
}
5656
}, {
5757
meta: {
58-
fixable: 'whitespace'
58+
fixable: 'whitespace',
59+
type: 'layout'
5960
}
6061
});

src/rules/noUndefinedTypes.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,4 +77,8 @@ export default iterateJsdoc(({
7777
}
7878
});
7979
});
80+
}, {
81+
meta: {
82+
type: 'suggestion'
83+
}
8084
});

src/rules/requireDescription.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,5 +43,6 @@ export default iterateJsdoc(({
4343
return noDefaults ?
4444
contexts :
4545
[...new Set([...defaultContexts, ...contexts])];
46-
}
46+
},
47+
type: 'suggestion'
4748
});

src/rules/requireDescriptionCompleteSentence.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,7 @@ export default iterateJsdoc(({
128128
});
129129
}, {
130130
meta: {
131-
fixable: 'code'
131+
fixable: 'code',
132+
type: 'suggestion'
132133
}
133134
});

src/rules/requireExample.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,4 +42,8 @@ export default iterateJsdoc(({
4242
report('Missing JSDoc @' + targetTagName + ' description.');
4343
}
4444
});
45+
}, {
46+
meta: {
47+
type: 'suggestion'
48+
}
4549
});

src/rules/requireHyphenBeforeParamDescription.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ export default iterateJsdoc(({
4545
});
4646
}, {
4747
meta: {
48-
fixable: 'code'
48+
fixable: 'code',
49+
type: 'layout'
4950
}
5051
});

src/rules/requireParam.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,4 +43,8 @@ export default iterateJsdoc(({
4343

4444
return false;
4545
});
46+
}, {
47+
meta: {
48+
type: 'suggestion'
49+
}
4650
});

src/rules/requireParamDescription.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,8 @@ export default iterateJsdoc(({
1111
report('Missing JSDoc @' + targetTagName + ' "' + jsdocParameter.name + '" description.', null, jsdocParameter);
1212
}
1313
});
14+
}, {
15+
meta: {
16+
type: 'suggestion'
17+
}
1418
});

src/rules/requireParamName.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,8 @@ export default iterateJsdoc(({
1111
report('There must be an identifier after @param ' + (jsdocParameter.type === '' ? 'type' : 'tag') + '.', null, jsdocParameter);
1212
}
1313
});
14+
}, {
15+
meta: {
16+
type: 'suggestion'
17+
}
1418
});

src/rules/requireParamType.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,8 @@ export default iterateJsdoc(({
1111
report('Missing JSDoc @' + targetTagName + ' "' + jsdocParameter.name + '" type.', null, jsdocParameter);
1212
}
1313
});
14+
}, {
15+
meta: {
16+
type: 'suggestion'
17+
}
1418
});

src/rules/requireReturns.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,4 +69,8 @@ export default iterateJsdoc(({
6969
)) {
7070
report('Missing JSDoc @' + tagName + ' declaration.');
7171
}
72+
}, {
73+
meta: {
74+
type: 'suggestion'
75+
}
7276
});

src/rules/requireReturnsCheck.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,4 +43,8 @@ export default iterateJsdoc(({
4343
if (utils.hasDefinedTypeReturnTag(tags[0]) && !utils.hasReturnValue()) {
4444
report('JSDoc @' + tagName + ' declaration present but return expression not available in function.');
4545
}
46+
}, {
47+
meta: {
48+
type: 'suggestion'
49+
}
4650
});

src/rules/requireReturnsDescription.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,4 +17,8 @@ export default iterateJsdoc(({
1717
report('Missing JSDoc @' + targetTagName + ' description.', null, jsdocTag);
1818
}
1919
});
20+
}, {
21+
meta: {
22+
type: 'suggestion'
23+
}
2024
});

src/rules/requireReturnsType.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,8 @@ export default iterateJsdoc(({
1111
report('Missing JSDoc @' + targetTagName + ' type.', null, jsdocTag);
1212
}
1313
});
14+
}, {
15+
meta: {
16+
type: 'suggestion'
17+
}
1418
});

src/rules/validTypes.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,4 +46,8 @@ export default iterateJsdoc(({
4646
validTypeParsing(tag.type);
4747
}
4848
});
49+
}, {
50+
meta: {
51+
type: 'suggestion'
52+
}
4953
});

0 commit comments

Comments
 (0)