Upgrade deps to latest versions and replace var with const/let
#35
Conversation
|
New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/[email protected], npm/[email protected], npm/[email protected] |
| const markdown = require('eslint-plugin-markdown') | ||
|
|
||
| module.exports = [ | ||
| ...neostandard({ |
There was a problem hiding this comment.
I had to migrate to neostandard because standard does not support eslint 9
There was a problem hiding this comment.
cc: @ctcpip @wesleytodd I know that there is long discussion on going about linters, can you confirm if we are align with your proposal for the "near future"? 🙏
| { | ||
| languageOptions: { | ||
| // See https://compat-table.github.io/compat-table/es2016plus/#node18_3 | ||
| ecmaVersion: 2022, |
There was a problem hiding this comment.
After reviewing this compatibility table, it looks like Node.js v18 fully supports ECMAScript 2022, with only a couple of RegExp flags not fully implemented.
Let me know if you think we should target a lower ECMAScript version.
| @@ -39,7 +41,7 @@ | |||
| "node": ">= 18" | |||
| }, | |||
| "scripts": { | |||
| "lint": "eslint --plugin markdown --ext js,md .", | |||
| "lint": "eslint --flag unstable_config_lookup_from_file .", | |||
There was a problem hiding this comment.
Only way I found to add the needed config for tests. This flag enables the new lookup behaviour starting in eslint v10.0.0,
Refs: #25