feat(scripts): drop lint (#5751)

Author: jmfrancois

.changeset/cozy-cases-sin.md

@@ -0,0 +1,5 @@
+---
+'@talend/scripts-core': major
+---
+
+feat: drop lint command from talend-scripts

.changeset/ready-snails-sneeze.md

@@ -0,0 +1,5 @@
+---
+'@talend/eslint-config': patch
+---
+
+fix: remove storybook eslint plugin from the list

tools/scripts-config-eslint/__tests__/config.test.js

@@ -1,8 +1,3 @@
-vi.mock('eslint-plugin-storybook', () => ({
-	configs: { 'flat/recommended': [] },
-	rules: {},
-}));
-
 vi.mock('eslint-plugin-mdx', () => ({
 	configs: { flat: {} },
 	rules: {},

tools/scripts-config-eslint/eslint.config.js

@@ -16,7 +16,6 @@ const talendPlugin = require('@talend/eslint-plugin');
 const angularPlugin = require('eslint-plugin-angular');
 const jestDomPlugin = require('eslint-plugin-jest-dom');
 const testingLibraryPlugin = require('eslint-plugin-testing-library');
-const storybookPlugin = require('eslint-plugin-storybook');
 const mdxPlugin = require('eslint-plugin-mdx');
 
 function tsConfig() {
@@ -219,5 +218,4 @@ module.exports = defineConfig([
 	...tsConfigs,
 	testConfig,
 	mdxConfig,
-	...storybookPlugin.configs['flat/recommended'],
 ]);

tools/scripts-config-eslint/package.json

@@ -39,7 +39,6 @@
     "eslint-plugin-prettier": "^5.5.5",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.0.1",
-    "eslint-plugin-storybook": "^10.2.10",
     "eslint-plugin-testing-library": "^7.16.0",
     "globals": "^15.14.0",
     "typescript-eslint": "^8.56.1"

tools/scripts-core/package.json

@@ -23,19 +23,11 @@
     "test:cov": "vitest run"
   },
   "peerDependencies": {
-    "@talend/scripts-config-react-webpack": ">=16.0.0",
-    "@talend/scripts-config-storybook-lib": ">=5.0.0",
-    "storybook": ">=7.4.0"
+    "@talend/scripts-config-react-webpack": ">=16.0.0"
   },
   "peerDependenciesMeta": {
     "@talend/scripts-config-react-webpack": {
       "optional": true
-    },
-    "@talend/scripts-config-storybook-lib": {
-      "optional": true
-    },
-    "storybook": {
-      "optional": true
     }
   },
   "dependencies": {
@@ -45,9 +37,7 @@
     "@talend/eslint-plugin": "^1.8.1",
     "@talend/scripts-utils": "^2.8.1",
     "@talend/scripts-config-babel": "^13.10.1",
-    "@talend/scripts-config-stylelint": "^4.5.1",
     "@talend/scripts-config-typescript": "^12.1.1",
-    "stylelint": "^17.4.0",
     "babel-loader": "^9.2.1",
     "cpx2": "^8.0.0",
     "fs-extra": "^10.1.0",

tools/scripts-core/src/config/webpack.config.js

@@ -1,5 +1,3 @@
-/* eslint-disable global-require */
-/* eslint-disable import/no-dynamic-require */
 /* eslint-disable no-console */
 import { merge } from 'webpack-merge';
 import * as utils from '@talend/scripts-utils';
@@ -19,7 +17,6 @@ export default async (env = {}) => {
 	const presetApi = getPresetApi();
 
 	let webpackConfigurations = [];
-	// eslint-disable-next-line import/no-extraneous-dependencies
 	const defaultConfig = await import('@talend/scripts-config-react-webpack');
 	webpackConfigurations = webpackConfigurations.concat(
 		defaultConfig.default(presetApi, { umd: env.umd }),

tools/scripts-core/src/index.js

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 
 /* eslint-disable no-console */
-import mergeReport from './scripts/lint-merge-report.js';
 import { getEnv } from './utils/env.js';
 import { printSeparator } from './utils/log.js';
 import { getPresetApi } from './utils/preset.js';
@@ -13,9 +12,6 @@ if (command === '--help' || command === '-h' || command === 'help') {
 	console.log(`Please use one of the following commands:
 * start
 * build
-* lint-merge-report
-* test
-* extends
 `);
 	process.exit(0);
 }
@@ -78,44 +74,8 @@ async function runScript() {
 switch (command) {
 	case 'build':
 	case 'start':
-	case 'test':
 		runScript(command, options);
 		break;
-	case 'lint-merge-report':
-		mergeReport(options);
-		break;
-	case 'lint':
-		console.log(
-			'The "lint" command has been removed. Please use "eslint ." directly with @talend/eslint-config',
-		);
-		process.exit(1);
-		break;
-	case 'build:lib':
-	case 'build:lib:umd':
-		console.log('This command do not exists anymore, please use just "build" command');
-		process.exit(-1);
-		break;
-	case 'lint:es':
-	case 'lint:style':
-		console.log('This command do not exists anymore, please use just "lint" command');
-		process.exit(-1);
-		break;
-	case 'test:ng':
-		console.log('This command do not exists anymore, please use just "test" command');
-		process.exit(-1);
-		break;
-	case 'upgrade:deps':
-		console.log(
-			'This command do not exists anymore, please use just "talend-upgrade-deps" binary from "@talend/upgrade-deps" package',
-		);
-		process.exit(-1);
-		break;
-	case 'publish:local':
-		console.log(
-			'This command do not exists anymore, please use just "talend-publish-local" bin from "@talend/scripts-publish-local" package',
-		);
-		process.exit(-1);
-		break;
 	default:
 		console.log(`Command ${command} not found.`);
 		process.exit(-1);