-
-
Notifications
You must be signed in to change notification settings - Fork 18
Breaking: drop node v10/v13/v15 (refs eslint/eslint#14023) #23
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Let’s make sure we sync these changes with the final changes in the eslint repo |
By "final changes", do you just mean whatever the latest is by the time this is merged. With a quick look, I didn't see a current issue or PR to add |
Oh, apologies, I guess it must be eslint/eslint#14023 as in this issue title. |
Yes, sorry, that’s the correct issue. Was replying on my phone :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
* Breaking: Switch to ESM * Chore: Sync engines changes with #23; add CI to reflect #23 * Fix: Add `dist` to `files` * New: Export `package.json` * Update package.json Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com> * Docs: Show CommonJS usage as well * Refactor: Align with espree re: ESM * Refactor: Switch from JSON to ESM * Chore: Add linting devDeps. * Refactor: Remove no longer needed `external` * Test: Add `commonjs.cjs` test * Refactor: Remove dupe `env` key * Breaking: Change to named exports * Update README.md Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com> * Refactor: freeze in keys file * Update package.json Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com> * Update .eslintrc.json Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com> * Update .eslintrc.json Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com> * Update .eslintrc.json Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com> * Test: Split cjs and js tests * Refactor: Avoid freezing unused array * Refactor: Stop exporting map file * Refactor: Drop `default` Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com>
No description provided.