Skip to content

Commit 509e711

Browse files
authored
Add test for tweaked rule jsdoc type comment (#1372)
1 parent 322c44e commit 509e711

File tree

88 files changed

+101
-261
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

88 files changed

+101
-261
lines changed

lib/rules/alias-model-in-controller.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,7 @@ const ember = require('../utils/ember');
77
// Controllers - Alias your model
88
//------------------------------------------------------------------------------
99

10-
/**
11-
* @type {import('eslint').Rule.RuleModule}
12-
*/
10+
/** @type {import('eslint').Rule.RuleModule} */
1311
module.exports = {
1412
meta: {
1513
type: 'suggestion',

lib/rules/avoid-leaking-state-in-ember-objects.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,7 @@ function isAllowed(value) {
4848
// (Don't use arrays or objects as default props)
4949
//------------------------------------------------------------------------------
5050

51-
/**
52-
* @type {import('eslint').Rule.RuleModule}
53-
*/
51+
/** @type {import('eslint').Rule.RuleModule} */
5452
module.exports = {
5553
meta: {
5654
type: 'problem',

lib/rules/avoid-using-needs-in-controllers.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,7 @@ const ember = require('../utils/ember');
66
// Ember object rule - Avoid using needs in controllers
77
//------------------------------------------------------------------------------
88

9-
/**
10-
* @type {import('eslint').Rule.RuleModule}
11-
*/
9+
/** @type {import('eslint').Rule.RuleModule} */
1210
module.exports = {
1311
meta: {
1412
type: 'suggestion',

lib/rules/classic-decorator-hooks.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,7 @@ const ERROR_MESSAGE_DESTROY_IN_NON_CLASSIC =
55
const ERROR_MESSAGE_CONSTRUCTOR_IN_CLASSIC =
66
'You cannot use the constructor in a classic class. If the class can be converted, you can convert it to remove all classic APIs and remove the @classic decorator, then switch to the constructor.';
77

8-
/**
9-
* @type {import('eslint').Rule.RuleModule}
10-
*/
8+
/** @type {import('eslint').Rule.RuleModule} */
119
module.exports = {
1210
ERROR_MESSAGE_INIT_IN_NON_CLASSIC,
1311
ERROR_MESSAGE_DESTROY_IN_NON_CLASSIC,

lib/rules/classic-decorator-no-classic-methods.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,7 @@ function disallowedMethodErrorMessage(name) {
1818
return `The this.${name}() method is a classic ember object method, and can't be used in octane classes. You can refactor this usage to use a utility version instead (e.g. get(this, 'foo')), or to use native/modern syntax instead. Alternatively, you can add the @classic decorator to this class to continue using classic APIs.`;
1919
}
2020

21-
/**
22-
* @type {import('eslint').Rule.RuleModule}
23-
*/
21+
/** @type {import('eslint').Rule.RuleModule} */
2422
module.exports = {
2523
disallowedMethodErrorMessage,
2624

lib/rules/closure-actions.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,7 @@ const types = require('../utils/types');
88

99
const ERROR_MESSAGE = 'Use closure actions';
1010

11-
/**
12-
* @type {import('eslint').Rule.RuleModule}
13-
*/
11+
/** @type {import('eslint').Rule.RuleModule} */
1412
module.exports = {
1513
meta: {
1614
type: 'suggestion',

lib/rules/computed-property-getters.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,7 @@ const PREVENT_GETTER_MESSAGE = 'Do not use a getter inside computed properties.'
1313
const ALWAYS_WITH_SETTER_MESSAGE =
1414
'Always define a getter inside computed properties when using a setter.';
1515

16-
/**
17-
* @type {import('eslint').Rule.RuleModule}
18-
*/
16+
/** @type {import('eslint').Rule.RuleModule} */
1917
module.exports = {
2018
meta: {
2119
type: 'suggestion',

lib/rules/jquery-ember-run.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,7 @@ const EMBER_RUNLOOP_FUNCTIONS = [
2929
'throttle',
3030
];
3131

32-
/**
33-
* @type {import('eslint').Rule.RuleModule}
34-
*/
32+
/** @type {import('eslint').Rule.RuleModule} */
3533
module.exports = {
3634
meta: {
3735
type: 'problem',

lib/rules/named-functions-in-promises.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,7 @@ const types = require('../utils/types');
66
// General rule - Use named functions defined on objects to handle promises
77
//------------------------------------------------------------------------------
88

9-
/**
10-
* @type {import('eslint').Rule.RuleModule}
11-
*/
9+
/** @type {import('eslint').Rule.RuleModule} */
1210
module.exports = {
1311
meta: {
1412
type: 'suggestion',

lib/rules/new-module-imports.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,7 @@ const GLOBALS = MAPPING.reduce((memo, exportDefinition) => {
2222
// General rule - Use "New Module Imports" from Ember RFC #176
2323
//------------------------------------------------------------------------------
2424

25-
/**
26-
* @type {import('eslint').Rule.RuleModule}
27-
*/
25+
/** @type {import('eslint').Rule.RuleModule} */
2826
module.exports = {
2927
meta: {
3028
type: 'problem',

0 commit comments

Comments
 (0)