lint tasks

Author: ricardo-devis-agullo

Diff for: package.json

@@ -6,8 +6,8 @@
   "types": "./dist/index.d.ts",
   "bin": "./dist/oc-cli.js",
   "scripts": {
-    "lint": "npx @biomejs/biome check src",
-    "lint:apply": "npx @biomejs/biome check --apply src",
+    "lint": "npx @biomejs/biome check src test tasks",
+    "lint:apply": "npx @biomejs/biome check --apply src test tasks",
     "prebuild": "rimraf dist",
     "build": "npm run lint && tsc && node tasks/build.js",
     "git-stage-and-push": "node tasks/git-stage-and-push.js",

Diff for: tasks/build.js

@@ -1,6 +1,3 @@
-/* eslint-disable @typescript-eslint/no-var-requires */
-'use strict';
-
 const fs = require('fs-extra');
 const ocClientBrowser = require('oc-client-browser');
 const log = require('./logger');
@@ -63,7 +60,7 @@ ocClientBrowser.getLibs((err, libs) => {
         );
         log.complete('Client has been built and packaged');
       })
-      .catch(err => {
+      .catch((err) => {
         log.error(err);
       });
   });

Diff for: tasks/changelog.js

@@ -1,16 +1,13 @@
-/* eslint-disable @typescript-eslint/no-var-requires */
-'use strict';
-
 const async = require('async');
-const fs = require('fs');
+const fs = require('node:fs');
 const path = require('node:path');
 const git = require('./git');
 
 module.exports = () =>
   new Promise((resolve, reject) => {
     const writeChangelog = (changelog, cb) => {
       let result = '## Change Log';
-      changelog.forEach(pr => {
+      changelog.forEach((pr) => {
         if (pr.changes.length > 0) {
           result += `\n\n### ${pr.tag.to}\n${pr.changes.join('\n')}`;
         }
@@ -31,7 +28,7 @@ module.exports = () =>
             next(err);
           });
         },
-        err => {
+        (err) => {
           if (err) {
             return reject(err);
           }

Diff for: tasks/git-stage-and-push.js

@@ -1,6 +1,3 @@
-/* eslint-disable @typescript-eslint/no-var-requires */
-'use strict';
-
 const changelog = require('./changelog');
 const packageJson = require('../package');
 const path = require('node:path');

Diff for: tasks/git.js

@@ -1,16 +1,13 @@
-/* eslint-disable @typescript-eslint/no-var-requires */
-'use strict';
-
 const path = require('node:path');
 const simpleGit = require('simple-git');
 const git = simpleGit(path.join(__dirname, '..'));
-const fs = require('fs');
+const fs = require('node:fs');
 
 const utils = {
-  formatPrs: commits => {
+  formatPrs: (commits) => {
     const result = [];
 
-    commits.forEach(commit => {
+    commits.forEach((commit) => {
       const commitMessages = commit.message.split('Merge pull request');
       const isPr = commitMessages.length > 1;
       const isSquashedPr = !!commit.message.match(/(.*?)\(#(.*?)\)/g);
@@ -47,7 +44,7 @@ const utils = {
 
     return result;
   },
-  getFirstCommitHash: cb => {
+  getFirstCommitHash: (cb) => {
     git.log(['--reverse'], (err, changes) => {
       if (err) {
         cb(err, null);

Diff for: tasks/logger.js

@@ -1,16 +1,12 @@
-/* eslint-disable @typescript-eslint/no-var-requires */
-'use strict';
-
 const chalk = require('chalk');
 const nodeEmoji = require('node-emoji');
 
-const log = (col, style, emoji) =>
-  function (msg) {
-    // eslint-disable-next-line no-console
-    console.log(
-      chalk[style][col](msg) + (emoji ? ` ${nodeEmoji.get(emoji)}` : '')
-    );
-  };
+const log = (col, style, emoji) => (msg) => {
+  // eslint-disable-next-line no-console
+  console.log(
+    chalk[style][col](msg) + (emoji ? ` ${nodeEmoji.get(emoji)}` : '')
+  );
+};
 
 module.exports = {
   complete: log('green', 'reset', 'thumbsup'),

Diff for: tasks/mochaTest.js

@@ -1,6 +1,3 @@
-/* eslint-disable @typescript-eslint/no-var-requires */
-'use strict';
-
 const async = require('async');
 const fs = require('fs-extra');
 const glob = require('glob');
@@ -25,32 +22,32 @@ if (argv.silent) {
 
 const componentsToPackage = fs
   .readdirSync(componentsFixturesPath)
-  .filter(x => x !== 'handlebars3-component');
+  .filter((x) => x !== 'handlebars3-component');
 
 const packageComponent = (componentName, done) =>
   oc.cli.package(
     {
       componentPath: path.join(componentsFixturesPath, componentName),
       compress: false
     },
-    err => done(err)
+    (err) => done(err)
   );
 
 const addTestSuite = (dir, done) =>
   glob(path.join(__dirname, '..', dir), (err, files) => {
-    files.forEach(file => mocha.addFile(file));
+    files.forEach((file) => mocha.addFile(file));
     done();
   });
 
-async.eachSeries(componentsToPackage, packageComponent, err => {
+async.eachSeries(componentsToPackage, packageComponent, (err) => {
   if (err) {
     log.error(`Error during test components packaging: ${err}`);
     process.exit(1);
   } else {
-    log.complete(`Test components packaged`);
+    log.complete('Test components packaged');
   }
 
   async.each(testDirs, addTestSuite, () =>
-    mocha.run(err => process.on('exit', () => process.exit(err)))
+    mocha.run((err) => process.on('exit', () => process.exit(err)))
   );
 });

Diff for: tasks/npm-publish.js

@@ -24,7 +24,7 @@ cmd.on('error', () => {
   log.fatal('npm publish failed');
 });
 
-cmd.on('close', code => {
+cmd.on('close', (code) => {
   if (code === 0) {
     log.complete('npm publish succeeded');
   }

Diff for: tasks/version.js

@@ -1,6 +1,3 @@
-/* eslint-disable @typescript-eslint/no-var-requires */
-'use strict';
-
 const fs = require('fs-extra');
 const log = require('./logger');
 const packageJson = require('../package');