ember-cli / ember-cli/eslint-plugin-ember

`require-computed-macros` rule autofixer can lose comments

Open
#754 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
JavaScript
Stars
263
Forks
214
Avg merge
30m
Merged PRs (30d)
5

Description

Need to update the autofixer for [require-computed-macros](https://github.com/ember-cli/eslint-plugin-ember/blob/master/docs/rules/require-computed-macros.md) to keep track of comments.

Before autofix:
```js
someComputedProperty: computed('someKey', function() {
// My comment
return this.someKey;
// My other comment
}),
```

After autofix (current, incorrect):
```js
someComputedProperty: computed.reads('someKey'),
```

After autofix (desired):
```js
// My comment
// My other comment
someComputedProperty: computed.reads('someKey'),
```

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.