Skip to content

Commit

Permalink
chore: Maintenance (#3)
Browse files Browse the repository at this point in the history
- Update dependencies
- Align setup with `template-javascript`
- Update `ci` and `npm-publish` workflows
- Fix `test:coverage:collect"` script

### BREAKING CHANGES

- Require Node.js version >= 18
- Update `rollup` to 3.x
  • Loading branch information
nikolay-borzov authored Feb 24, 2023
1 parent 958d061 commit 1f89e48
Show file tree
Hide file tree
Showing 9 changed files with 4,865 additions and 4,742 deletions.
2 changes: 1 addition & 1 deletion .browserslistrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
node 14.16
node 18
23 changes: 10 additions & 13 deletions .eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,11 +1,6 @@
module.exports = {
root: true,

parser: '@babel/eslint-parser',
parserOptions: {
sourceType: 'module',
},

ignorePatterns: [
'**/*.*',
'!**/*.js',
Expand All @@ -14,25 +9,32 @@ module.exports = {
'node_modules',
'dist',
],

parser: '@babel/eslint-parser',
parserOptions: {
sourceType: 'module',
},

env: {
node: true,
},

extends: [
'eslint:recommended',
'plugin:jsdoc/recommended',
'plugin:n/recommended',
'plugin:unicorn/recommended',
'prettier-standard/prettier-file',
],

settings: {
jsdoc: {
mode: 'typescript',
},
},

// Keep rules grouped by plugin and sorted alphabetically
rules: {
'object-shorthand': ['error', 'always'],

'padding-line-between-statements': [
'error',
/* Empty line after import */
Expand Down Expand Up @@ -63,9 +65,6 @@ module.exports = {
'jsdoc/require-returns-description': 'off',
'jsdoc/require-returns': 'off',

'jsdoc/require-param-description': 'off',
'jsdoc/require-returns-description': 'off',

/* eslint-plugin-unicorn */

// I like reduce
Expand Down Expand Up @@ -103,9 +102,7 @@ module.exports = {

/* eslint-plugin-n (node) */

'n/no-unsupported-features/es-syntax': [
'error',
],
'n/no-unsupported-features/es-syntax': ['error'],
},

overrides: [
Expand Down
7 changes: 3 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,19 +8,18 @@ on:

jobs:
build:

strategy:
matrix:
node-version: [14.x, 16.x]
node: [18, 19]
os: [ windows-latest, ubuntu-latest, macos-latest ]

runs-on: ${{ matrix.os }}

steps:
- run: git config --global core.autocrlf input
- uses: actions/checkout@v2
- uses: actions/checkout@v3
- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v1
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}
- run: npm ci
Expand Down
16 changes: 9 additions & 7 deletions .github/workflows/npm-publish.yml
Original file line number Diff line number Diff line change
@@ -1,28 +1,30 @@
name: Publish to NPM

on:
workflow_dispatch:
release:
types: [created]
types: [published]

jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 14
node-version: 18
cache: 'npm'
- run: npm ci
- run: npm test

publish-npm:
needs: build
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 14
node-version: 18
registry-url: https://registry.npmjs.org/
- run: npm ci
- run: npm publish
Expand Down
80 changes: 64 additions & 16 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,26 +1,74 @@
# rollup-plugin-eslint-bundle changelog
# Changelog

<!--
### Added
### Changed
### Deprecated
### Removed
### Fixed
### Security
### BREAKING CHANGES
-->

## 8.0.0 - 2023-02-25

### Changed

- Update dependencies

### BREAKING CHANGES

- Require Node.js version >= 18
- Update `rollup` to 3.x

## 7.0.0 - 2022-05-08
* Switch to ES modules. But still provide CJS bundle in `dist`
* Update dependencies
* Require Node.js version >= 14
* Update ESLint peer dependency to >=8.x

### Changed

- Update dependencies

### BREAKING CHANGES

- Switch to ES modules. But still provide CJS bundle in `dist`
- Require Node.js version >= 14
- Update `eslint` peer dependency to >=8.x

## 6.0.0 - 2021-01-24
* Rewrite from scratch for latest `rollup` and `ESLint`
* Replace `jest` with `ava`
* Require Node.js version >= 12
* Set `rollup` and `eslint` as peer dependencies
* Generate TypeScript declaration files.

### Added

- Generate TypeScript declaration files.

### Changed

- Replace `jest` with `ava`
- Set `rollup` and `eslint` as peer dependencies

### BREAKING CHANGES

- Rewrite from scratch for latest `rollup` and `ESLint`
- Require Node.js version >= 12

## 5.0.2 - 2018-11-02
* Update dependencies

### Changed

- Update dependencies

## 5.0.1 - 2018-07-15
* Fix for case when fix doesn't produce output
* Fixes for ESLint 5.x
* Disable tests for source mapping. For some reason rollup returns the same 'mappings' for fixed and not fixed code even though the plugin produces different source maps
* Update dependencies

### Changed

- Disable tests for source mapping. For some reason rollup returns the same 'mappings' for fixed and not fixed code even though the plugin produces different source maps
- Update dependencies

### Fixed

- Fix for case when fix doesn't produce output
- Fixes for ESLint 5.x

## 5.0.0 - 2018-01-21
* Rewrite to verify and fix the bundle instead of imported files

### BREAKING CHANGES

- Rewrite to verify and fix the bundle instead of imported files
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

## Requirements

This plugin requires an LTS Node version (v14.13.1+), Rollup v2.x and ESLint >=8.x
This plugin requires an Node.js v18.x, Rollup v3.x and ESLint v8.x

## Install

Expand Down
Loading

0 comments on commit 1f89e48

Please sign in to comment.