ryanzim/eslint-config-problems

An eslint config that catches problems in your code, without checking style. For use with prettier.

eslint-config-problems's Issues

Missing node8 subconfig

I installed the new 3.0 version and try to run eslint. It says that it cannot find module eslint-config-problems/node8 when using the node10 subconfig.

The node10.js file references the removed node8 subconfig:

eslint-config-problems/node10.js

Line 3 in 39ecc6a

extends: 'problems/node8',

How should I solve this?

Add checkForEach

https://eslint.org/docs/latest/rules/array-callback-return#checkforeach

Merge renovate bot's updated eslint dependency

Hi!
Please merge the dependency update so that your config would support eslint v6.
Thanks. :)

Rule: strict

Hi,

I see that the rule strict is enabled by your config. I find that strange, as writing "use strict" in all my source files when writing a module in 2019 is unnecessary, especially that Module code is always strict mode code.

10.2.1 Strict Mode Code
[...]

• Module code is always strict mode code.

[...]

Shouldn't this rule be opt-in when writing code for the browser instead of opt-out for node.js? I mean, most of the time we use some kind of bundler for browser code which puts the "use strict" there anyway.

Enable logical-assignment-operators?

Should we enable https://eslint.org/docs/latest/rules/logical-assignment-operators on Node 16+?

Logically, if we enable it, we should perhaps also enable https://eslint.org/docs/latest/rules/operator-assignment.

However, this whole thing might be too stylistic for eslint-config-problems.

Dependency Dashboard

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• Update dependency eslint to ~8.57.0
• Update dependency eslint to v9

Detected dependencies

npm

package.json

• doctoc ^2.0.0
• eslint ~8.38.0
• lodash.defaultsdeep ^4.6.0
• prettier ^3.0.0
• eslint ^8.27.0

---

• Check this box to trigger a request for Renovate to run again on this repository

`async`/`await` support?

I love the idea behind this config, but it's complaining about async / await in my Node 8 code.