Skip to content

Commit 4f0a7e5

Browse files
committed
docs: add 'Help Wanted' section to docs for rules that are missing native JS class support
1 parent e8ccb5d commit 4f0a7e5

19 files changed

+114
-0
lines changed

docs/rules/alias-model-in-controller.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,3 +36,9 @@ export default Controller.extend({
3636
pets: reads('model.pets')
3737
});
3838
```
39+
40+
## Help Wanted
41+
42+
| Issue | Link |
43+
| :-- | :-- |
44+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/avoid-using-needs-in-controllers.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,3 +25,9 @@ export default Component.extend({
2525
newComments: alias('comments.newest')
2626
});
2727
```
28+
29+
## Help Wanted
30+
31+
| Issue | Link |
32+
| :-- | :-- |
33+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/computed-property-getters.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -100,3 +100,9 @@ Ember.Object.extend({
100100
})
101101
});
102102
```
103+
104+
## Help Wanted
105+
106+
| Issue | Link |
107+
| :-- | :-- |
108+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/no-arrow-function-computed-properties.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,3 +42,9 @@ This rule takes an optional object containing:
4242

4343
* [Arrow function spec](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions)
4444
* [Computed property spec](https://api.emberjs.com/ember/release/classes/ComputedProperty)
45+
46+
## Help Wanted
47+
48+
| Issue | Link |
49+
| :-- | :-- |
50+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/no-capital-letters-in-routes.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,3 +29,9 @@ this.route('sign-up');
2929
* [no-unnecessary-route-path-option](no-unnecessary-route-path-option.md)
3030
* [route-path-style](route-path-style.md)
3131
* [routes-segments-snake-case](routes-segments-snake-case.md)
32+
33+
## Help Wanted
34+
35+
| Issue | Link |
36+
| :-- | :-- |
37+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/no-deeply-nested-dependent-keys-with-each.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,9 @@ owners: computed('[email protected]', function() {
3131
## Further Reading
3232

3333
* See the [documentation](https://guides.emberjs.com/release/object-model/computed-properties-and-aggregate-data/) for Ember computed properties with `@each`
34+
35+
## Help Wanted
36+
37+
| Issue | Link |
38+
| :-- | :-- |
39+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/no-duplicate-dependent-keys.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,9 @@ computed('foo.{bar,baz,qux}', 'bar.foo', function() {
3131
//...
3232
})
3333
```
34+
35+
## Help Wanted
36+
37+
| Issue | Link |
38+
| :-- | :-- |
39+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/no-empty-attrs.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,9 @@ export default Model.extend({
3131
```
3232

3333
In case you need a custom behavior, it's good to write your own [transform](http://emberjs.com/api/data/classes/DS.Transform.html).
34+
35+
## Help Wanted
36+
37+
| Issue | Link |
38+
| :-- | :-- |
39+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/no-side-effects.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,3 +38,9 @@ export default Component.extend({
3838
})
3939
});
4040
```
41+
42+
## Help Wanted
43+
44+
| Issue | Link |
45+
| :-- | :-- |
46+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/no-unnecessary-service-injection-argument.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,3 +41,9 @@ export default Component.extend({
4141

4242
* Ember [Services](https://guides.emberjs.com/release/applications/services/) guide
4343
* Ember [inject](https://emberjs.com/api/ember/release/functions/@ember%2Fservice/inject) function spec
44+
45+
## Help Wanted
46+
47+
| Issue | Link |
48+
| :-- | :-- |
49+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/no-volatile-computed-properties.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,3 +35,9 @@ const Person = EmberObject.extend({
3535
* [Deprecation list](https://deprecations.emberjs.com/v3.x/#toc_computed-property-volatile)
3636
* [Volatile spec](https://api.emberjs.com/ember/release/classes/ComputedProperty/methods/volatile?anchor=volatile)
3737
* [Computed property spec](https://api.emberjs.com/ember/release/classes/ComputedProperty)
38+
39+
## Help Wanted
40+
41+
| Issue | Link |
42+
| :-- | :-- |
43+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/order-in-components.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -129,3 +129,9 @@ export default Component.extend({
129129
}
130130
});
131131
```
132+
133+
## Help Wanted
134+
135+
| Issue | Link |
136+
| :-- | :-- |
137+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/order-in-controllers.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -98,3 +98,9 @@ export default Controller.extend({
9898
},
9999
});
100100
```
101+
102+
## Help Wanted
103+
104+
| Issue | Link |
105+
| :-- | :-- |
106+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/order-in-models.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,3 +71,9 @@ export default Model.extend({
7171
shape: attr('string')
7272
});
7373
```
74+
75+
## Help Wanted
76+
77+
| Issue | Link |
78+
| :-- | :-- |
79+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/order-in-routes.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,3 +124,9 @@ export default Route.extend({
124124
},
125125
});
126126
```
127+
128+
## Help Wanted
129+
130+
| Issue | Link |
131+
| :-- | :-- |
132+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/require-computed-macros.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,3 +89,9 @@ export default Component.extend({
8989

9090
* [Guide](https://guides.emberjs.com/release/object-model/computed-properties/) for computed properties
9191
* [Spec](http://api.emberjs.com/ember/release/modules/@ember%2Fobject#functions-computed) for computed property macros
92+
93+
## Help Wanted
94+
95+
| Issue | Link |
96+
| :-- | :-- |
97+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/require-computed-property-dependencies.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,3 +49,9 @@ This rule takes an optional object containing:
4949
## References
5050

5151
* [Guide](https://guides.emberjs.com/release/object-model/computed-properties/) for computed properties
52+
53+
## Help Wanted
54+
55+
| Issue | Link |
56+
| :-- | :-- |
57+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/require-return-from-computed.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,3 +73,9 @@ To avoid false positives from relying on implicit returns in some code branches,
7373
* [consistent-return] from eslint
7474

7575
[consistent-return]: https://eslint.org/docs/rules/consistent-return
76+
77+
## Help Wanted
78+
79+
| Issue | Link |
80+
| :-- | :-- |
81+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

docs/rules/use-brace-expansion.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,3 +21,9 @@ fullName: computed('user.{firstName,lastName}', {
2121
// Code
2222
})
2323
```
24+
25+
## Help Wanted
26+
27+
| Issue | Link |
28+
| :-- | :-- |
29+
| :x: Missing native JavaScript class support | [#560](https://github.com/ember-cli/eslint-plugin-ember/issues/560) |

0 commit comments

Comments
 (0)