search

AtomLinter / linter-eslint-node

ESLint plugin for Atom/Pulsar Linter (v8 and above)
https://web.pulsar-edit.dev/packages/linter-eslint-node
MIT License
4 stars 3 forks source link

chore!: Make eslint@8 the minimum supported version for this package #28

Closed scagood closed 1 year ago

scagood commented 2 years ago

See #25 #27

This simply bumps the lowest supported version of eslint to v8.

~TODO: Make the same change in the https://github.com/AtomLinter/linter-eslint~

scagood commented 2 years ago

When checking on the linter-eslint repo I can see that there is no change needed in terms of the alert: https://github.com/AtomLinter/linter-eslint/blob/master/src/worker-helpers.js#L138-L141

scagood commented 2 years ago

Force push due to: https://github.com/AtomLinter/linter-eslint-node/runs/6786870142?check_suite_focus=true

scagood commented 2 years ago

🤔 should lint be style?

UziTech commented 2 years ago

I usually stick with chore(lint): but it's not a big deal semantic release will see lint: and chore(lint): as the same. CommitLint is more picky than it needs to be.