Skip to content
This repository was archived by the owner on Oct 10, 2022. It is now read-only.

Conversation

@ehmicky
Copy link
Contributor

@ehmicky ehmicky commented Dec 8, 2020

This adds eslint-config-node.

Many rule are currently disabled, in an effort to add this incrementally.

@ehmicky ehmicky added the type: chore work needed to keep the product and development running smoothly label Dec 8, 2020
@ehmicky ehmicky requested a review from erezrokah December 8, 2020 19:16
@ehmicky ehmicky self-assigned this Dec 8, 2020
matrix:
os: [ubuntu-latest, macOS-latest, windows-latest]
node-version: [8.3.0, 14]
node-version: [8.17.0, 14.x]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Align with Netlify Build. We're also removing Node 8 support soon.

const client = new NetlifyAPI('1234myAccessToken')
const sites = await client.listSites()
return sites
}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top-level await still confuses ESLint.


```js
{
const opts = {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise, ESLint seems to fail at parsing this.

@ehmicky ehmicky force-pushed the chore/eslint-config-node branch from 678b3e4 to f040593 Compare December 8, 2020 19:26
@erezrokah erezrokah merged commit 1676d3e into master Dec 9, 2020
@erezrokah erezrokah deleted the chore/eslint-config-node branch December 9, 2020 10:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

type: chore work needed to keep the product and development running smoothly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants