Skip to content

Commit

Permalink
chore: replace eslint-config-semistandard with eslint-config-standard (
Browse files Browse the repository at this point in the history
…nodejs#4586)

eslint-config-semistandard hasn't been published in a year as of this
writing and does not install with the version of eslint that we use
without --force or --legacy-peer-deps.

Replace it with the up-to-date eslint-config-standard, and change the
semi rule in .eslintrc to have the same impact. This also disables
two (new?) rules that now flag errors in our markdown files. These
can be investigated later or not. It also makes a small change to
release-post.js so it complies with a rule about using object
shorthand when possible.

Fixes: nodejs#4585
  • Loading branch information
Trott authored May 3, 2022
1 parent 6c16990 commit 56c7570
Show file tree
Hide file tree
Showing 4 changed files with 202 additions and 37 deletions.
7 changes: 5 additions & 2 deletions .eslintrc
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"extends": [
"eslint:recommended",
"semistandard",
"standard",
"prettier"
],
"plugins": [
Expand Down Expand Up @@ -29,6 +29,8 @@
},
"rules": {
"eqeqeq": "off",
"n/no-deprecated-api": "off",
"n/handle-callback-err": "off",
"no-const-assign": "off",
"no-undef": "off",
"no-unused-expressions": "off",
Expand All @@ -42,7 +44,8 @@
"singleQuote": true,
"trailingComma": "none"
}
]
],
"semi": ["error", "always"]
}
}
]
Expand Down
228 changes: 195 additions & 33 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@
"cross-env": "^7.0.3",
"eslint": "^8.14.0",
"eslint-config-prettier": "^8.5.0",
"eslint-config-semistandard": "^16.0.0",
"eslint-config-standard": "^17.0.0",
"eslint-plugin-import": "^2.26.0",
"eslint-plugin-markdown": "^2.2.1",
"eslint-plugin-node": "^11.1.0",
Expand Down
2 changes: 1 addition & 1 deletion scripts/release-post.js
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ function findAuthorLogin(version, section) {

function urlOrComingSoon(binary) {
const url = binary.url.replace('nodejs.org', 'direct.nodejs.org');
return sendRequest({ url: url, method: 'HEAD' }).then(
return sendRequest({ url, method: 'HEAD' }).then(
() => `${binary.title}: ${binary.url}`,
() => {
console.log(`\x1B[32m "${binary.title}" is Coming soon...\x1B[39m`);
Expand Down

0 comments on commit 56c7570

Please sign in to comment.