eslint-plugin-amo
Advanced tools
Comparing version 1.8.0 to 1.9.0
{ | ||
"name": "eslint-plugin-amo", | ||
"version": "1.8.0", | ||
"version": "1.9.0", | ||
"description": "ESLint plugin for AMO", | ||
@@ -5,0 +5,0 @@ "keywords": [ |
@@ -63,2 +63,3 @@ # eslint-plugin-amo | ||
- [`one-top-level-describe-per-test`](#one-top-level-describe-per-test) | ||
- [`only-log-strings`](#only-log-strings) | ||
- [`redux-app-state`](#redux-app-state) | ||
@@ -149,2 +150,18 @@ - [`sort-destructured-props`](#sort-destructured-props) | ||
### `only-log-strings` | ||
Ensure we do not log full objects: | ||
```js | ||
// BAD | ||
log.info('response:', response); | ||
// GOOD | ||
log.info('this is a log message'); | ||
log.debug(oneLine`A very long string message`); | ||
_log.warn(`request ID: ${requestId}`); | ||
``` | ||
:bulb: We enforce this rule because of the following issue: https://github.com/mozilla/addons-frontend/issues/6512. | ||
### `redux-app-state` | ||
@@ -151,0 +168,0 @@ |
58808
30
904
248