chore: remove (deprecated) eslint-config-google

.eslintignore

@@ -23,3 +23,4 @@
 ##
 
 /dist
+/test/fixtures

.eslintrc

@@ -1,18 +1,42 @@
 {
   "extends": [
     "eslint:recommended",
-    "google"
+    "airbnb-base"
   ],
   "parserOptions": {
-    "ecmaVersion": 2020
+    "ecmaVersion": "latest"
   },
   "env": {
     "es6": true,
     "node": true
   },
   "rules": {
     "max-len": [2, 150, 2],
+
+    "import/prefer-default-export": "off",
+
+    "import/no-extraneous-dependencies": ["error", {
+      "devDependencies": [
+        "scripts/**/*.js",
+        "test/**/*.js",
+        "gulpfile.js"
+      ]
+    }],
+
+    "operator-linebreak": "off",
+    "consistent-return": "off",
+    "no-underscore-dangle": "off",
+    "no-plusplus": "off",
+    "quote-props": ["error", "consistent-as-needed"],
+    "no-use-before-define": ["error", {
+      "functions": false,
+      "classes": true,
+      "variables": true,
+      "allowNamedExports": true
+    }],
+
     "no-console": [0],
+
     "valid-jsdoc": [2, {
       "requireReturn": true,
       "requireParamDescription": true,

package.json

@@ -57,9 +57,11 @@
     "@rollup/plugin-virtual": "3.0.2",
     "@typescript-eslint/eslint-plugin": "7.10.0",
     "@typescript-eslint/parser": "7.10.0",
+    "ansi-colors": "4.1.3",
     "babel-plugin-add-module-exports": "1.0.4",
     "eslint": "8.57.0",
-    "eslint-config-google": "0.14.0",
+    "eslint-config-airbnb-base": "15.0.0",
+    "eslint-plugin-import": "2.29.1",
     "fancy-log": "2.0.0",
     "fs-extra": "11.2.0",
     "globalthis": "1.0.4",

scripts/build/index.js

@@ -29,8 +29,8 @@ const rollupConfig = require('./rollup.config');
 const config = require('../config');
 
 module.exports = gulp.series(
-    buildOutput,
-    copyTypings,
+  buildOutput,
+  copyTypings,
 );
 
 // eslint-disable-next-line require-jsdoc

scripts/build/rollup.config.js

@@ -34,6 +34,7 @@ const prettier = require('rollup-plugin-prettier');
 
 const config = require('../config');
 const pkg = require('../../package.json');
+
 const license = fs.readFileSync(path.join(config.root, 'LICENSE'), 'utf-8');
 
 module.exports = {

scripts/changelog/index.js

@@ -28,8 +28,8 @@ const conventionalChangelog = require('gulp-conventional-changelog');
 const config = require('../config');
 
 module.exports = function changelog() {
-  const changelog = path.join(config.root, 'CHANGELOG.md');
-  return gulp.src(changelog, {buffer: false})
-      .pipe(conventionalChangelog({releaseCount: 0}))
-      .pipe(gulp.dest(config.root));
+  const changelogFile = path.join(config.root, 'CHANGELOG.md');
+  return gulp.src(changelogFile, { buffer: false })
+    .pipe(conventionalChangelog({ releaseCount: 0 }))
+    .pipe(gulp.dest(config.root));
 };

scripts/clean/index.js

@@ -22,7 +22,7 @@
  * SOFTWARE.
  */
 
-const {rimraf} = require('rimraf');
+const { rimraf } = require('rimraf');
 const config = require('../config');
 
 module.exports = function clean() {

scripts/config.js

@@ -23,6 +23,7 @@
  */
 
 const path = require('node:path');
+
 const ROOT = path.join(__dirname, '..');
 
 module.exports = {

scripts/lint/index.js

@@ -52,8 +52,12 @@ module.exports = function lint() {
     log.debug(`  ${input}`)
   ));
 
-  const {ESLint} = require('eslint');
+  // eslint-disable-next-line global-require
+  const { ESLint } = require('eslint');
+
+  // eslint-disable-next-line global-require
   const fancyLog = require('fancy-log');
+
   const eslint = new ESLint({
     errorOnUnmatchedPattern: false,
   });

scripts/release/index.js

@@ -64,7 +64,7 @@ function bumpVersion(type) {
  */
 function prepareNextRelease() {
   return run(`git rm -r "${config.dist}"`).then(() => (
-    run(`git commit -m 'release: prepare next release'`)
+    run("git commit -m 'release: prepare next release'")
   ));
 }
 
@@ -86,8 +86,8 @@ function createReleaseTask(level) {
   }
 
   return gulp.series(
-      doRelease,
-      prepareNextRelease,
+    doRelease,
+    prepareNextRelease,
   );
 }
 

src/dependency.js

@@ -23,8 +23,8 @@
  */
 
 import _ from 'lodash';
-import {EOL} from './eol.js';
-import {Person} from './person.js';
+import { EOL } from './eol';
+import { Person } from './person';
 
 /**
  * Dependency structure.
@@ -95,9 +95,9 @@ export class Dependency {
     }
 
     if (this.contributors.length > 0) {
-      lines.push(`Contributors:`);
+      lines.push('Contributors:');
       lines.push(
-          ...this.contributors.map((contributor) => `  ${contributor.text()}`),
+        ...this.contributors.map((contributor) => `  ${contributor.text()}`),
       );
     }
 

src/generate-block-comment.js

@@ -38,9 +38,9 @@ export function generateBlockComment(text, commentStyle) {
 
   if (commentStyle) {
     options.style = new commenting.Style(
-        commentStyle.body,
-        commentStyle.start,
-        commentStyle.end,
+      commentStyle.body,
+      commentStyle.start,
+      commentStyle.end,
     );
   }
 

src/index.d.ts

@@ -22,7 +22,7 @@
  * SOFTWARE.
  */
 
-import type {Plugin} from 'rollup';
+import type { Plugin } from 'rollup';
 
 type FilePath = string;
 type FileEncoding = string;
@@ -198,7 +198,7 @@ type SpdxId = string;
 /**
  * Function checking dependency license validity.
  */
-type ThirdPartyDependencyValidatorFn = (Dependency: Dependency) => boolean;
+type ThirdPartyDependencyValidatorFn = (dependency: Dependency) => boolean;
 
 type ThirdPartyValidator = SpdxId | ThirdPartyDependencyValidatorFn;
 

src/index.js

@@ -22,7 +22,7 @@
  * SOFTWARE.
  */
 
-import {licensePlugin} from './license-plugin.js';
+import { licensePlugin } from './license-plugin';
 
 /**
  * Create rollup plugin compatible with rollup >= 1.0.0
@@ -62,7 +62,7 @@ export default function rollupPluginLicense(options = {}) {
       }
 
       plugin.scanDependencies(
-          dependencies,
+        dependencies,
       );
 
       return plugin.prependBanner(code, outputOptions.sourcemap !== false);

src/license-plugin-option.js

@@ -23,10 +23,10 @@
  */
 
 import _ from 'lodash';
-import {PLUGIN_NAME} from './license-plugin-name.js';
-import {validators} from './schema-validators.js';
-import {validateSchema} from './schema-validator.js';
-import {formatPath} from './format-path.js';
+import { PLUGIN_NAME } from './license-plugin-name';
+import { validators } from './schema-validators';
+import { validateSchema } from './schema-validator';
+import { formatPath } from './format-path';
 
 /**
  * The option object schema.
@@ -152,7 +152,7 @@ function validateOptions(options) {
 
   if (messages.length > 0) {
     throw new Error(
-        `[${PLUGIN_NAME}] -- Error during validation of option object: ${messages.join(' ; ')}`,
+      `[${PLUGIN_NAME}] -- Error during validation of option object: ${messages.join(' ; ')}`,
     );
   }
 }

src/license-plugin.js

@@ -24,19 +24,19 @@
 
 import fs from 'fs';
 import path from 'path';
-import {mkdirp} from 'mkdirp';
+import { mkdirp } from 'mkdirp';
 import _ from 'lodash';
 import moment from 'moment';
 import MagicString from 'magic-string';
 import packageNameRegex from 'package-name-regex';
 
-import {Dependency} from './dependency.js';
-import {generateBlockComment} from './generate-block-comment.js';
-import {licensePluginOptions} from './license-plugin-option.js';
-import {licenseValidator} from './license-validator';
-import {readFile} from './read-file';
-import {PLUGIN_NAME} from './license-plugin-name.js';
-import {EOL} from './eol.js';
+import { Dependency } from './dependency';
+import { generateBlockComment } from './generate-block-comment';
+import { licensePluginOptions } from './license-plugin-option';
+import { licenseValidator } from './license-validator';
+import { readFile } from './read-file';
+import { PLUGIN_NAME } from './license-plugin-name';
+import { EOL } from './eol';
 
 /**
  * Pre-Defined comment style:
@@ -103,9 +103,11 @@ class LicensePlugin {
     this._options = options;
     this._cwd = this._options.cwd || process.cwd();
     this._dependencies = new Map();
-    this._pkg = require(path.join(this._cwd, 'package.json'));
     this._debug = this._options.debug || false;
 
+    // eslint-disable-next-line import/no-dynamic-require, global-require
+    this._pkg = require(path.join(this._cwd, 'package.json'));
+
     // SourceMap can now be disable/enable on the plugin.
     this._sourcemap = this._options.sourcemap !== false;
 
@@ -139,6 +141,7 @@ class LicensePlugin {
    */
   scanDependency(id) {
     if (id.startsWith('\0')) {
+      // eslint-disable-next-line no-param-reassign
       id = id.replace(/^\0/, '');
       this.debug(`scanning internal module ${id}`);
     }
@@ -186,15 +189,14 @@ class LicensePlugin {
 
         // Read `package.json` file
         const pkgJson = JSON.parse(
-            fs.readFileSync(pkgPath, 'utf-8'),
+          fs.readFileSync(pkgPath, 'utf-8'),
         );
 
         // We are probably in a package.json specifying the type of package (module, cjs).
         // Nevertheless, if the package name is not defined, we must not use this `package.json` descriptor.
         const license = pkgJson.license || pkgJson.licenses;
         const hasLicense = license && license.length > 0;
-        const name = pkgJson.name;
-        const version = pkgJson.version;
+        const { name, version } = pkgJson;
         const isValidPackageName = name && packageNameRegex.test(name);
         if ((isValidPackageName && version) || hasLicense) {
           // We found it!
@@ -270,7 +272,7 @@ class LicensePlugin {
     // will be used to generate the sourcemap.
     const magicString = new MagicString(code);
 
-    const banner = this._options.banner;
+    const { banner } = this._options;
     const content = this._readBanner(banner);
     if (content) {
       magicString.prepend(EOL);
@@ -318,7 +320,7 @@ class LicensePlugin {
    * @return {void}
    */
   scanThirdParties() {
-    const thirdParty = this._options.thirdParty;
+    const { thirdParty } = this._options;
     if (!thirdParty) {
       return;
     }
@@ -333,12 +335,12 @@ class LicensePlugin {
       return;
     }
 
-    const allow = thirdParty.allow;
+    const { allow, output } = thirdParty;
+
     if (allow) {
       this._scanLicenseViolations(outputDependencies, allow);
     }
 
-    const output = thirdParty.output;
     if (output) {
       this._exportThirdParties(outputDependencies, output);
     }
@@ -398,7 +400,7 @@ class LicensePlugin {
       throw new Error(`[${this.name}] -- Cannot find banner content, please specify an inline content, or a path to a file`);
     }
 
-    const file = content.file;
+    const { file } = content;
     const encoding = content.encoding || 'utf-8';
 
     this.debug(`prepend banner from file: ${file}`);
@@ -434,7 +436,13 @@ class LicensePlugin {
     const pkg = this._pkg;
     const dependencies = [...this._dependencies.values()];
     const data = banner.data ? _.result(banner, 'data') : {};
-    const text = tmpl({_, moment, pkg, dependencies, data});
+    const text = tmpl({
+      _,
+      moment,
+      pkg,
+      dependencies,
+      data,
+    });
 
     // Compute comment style to use.
     const style = _.has(banner, 'commentStyle') ? banner.commentStyle : computeDefaultCommentStyle(text);
@@ -523,9 +531,10 @@ class LicensePlugin {
    * @return {void}
    */
   _handleLicenseViolation(dependency, fail) {
-    const message =
+    const message = (
       `Dependency "${dependency.name}" has a license (${dependency.license}) which is not compatible with ` +
-      `requirement, looks like a license violation to fix.`;
+      'requirement, looks like a license violation to fix.'
+    );
 
     if (!fail) {
       this.warn(message);
@@ -565,7 +574,7 @@ class LicensePlugin {
     // Default is to export to given file.
 
     // Allow custom formatting of output using given template option.
-    const template = _.isString(output.template) ? (dependencies) => _.template(output.template)({dependencies, _, moment}) : output.template;
+    const template = _.isString(output.template) ? (dependencies) => _.template(output.template)({ dependencies, _, moment }) : output.template;
     const defaultTemplate = (dependencies) => (
       dependencies.length === 0 ? 'No third parties dependencies' : dependencies.map((d) => d.text()).join(`${EOL}${EOL}---${EOL}${EOL}`)
     );
@@ -596,6 +605,6 @@ class LicensePlugin {
  */
 export function licensePlugin(options) {
   return new LicensePlugin(
-      licensePluginOptions(options),
+    licensePluginOptions(options),
   );
 }