Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/cozy-cases-sin.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@talend/scripts-core': major
---

feat: drop lint command from talend-scripts
5 changes: 5 additions & 0 deletions .changeset/ready-snails-sneeze.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@talend/eslint-config': patch
---

fix: remove storybook eslint plugin from the list
5 changes: 0 additions & 5 deletions tools/scripts-config-eslint/__tests__/config.test.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,3 @@
vi.mock('eslint-plugin-storybook', () => ({
configs: { 'flat/recommended': [] },
rules: {},
}));

vi.mock('eslint-plugin-mdx', () => ({
configs: { flat: {} },
rules: {},
Expand All @@ -10,7 +5,7 @@

describe('eslint config', () => {
it('should require', () => {
// eslint-disable-next-line global-require

Check warning on line 8 in tools/scripts-config-eslint/__tests__/config.test.js

View workflow job for this annotation

GitHub Actions / ESLint Report Analysis

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

[null] Unused eslint-disable directive (no problems were reported from 'global-require').
const config = require('../index');
expect(config).toBeDefined();
});
Expand Down
2 changes: 0 additions & 2 deletions tools/scripts-config-eslint/eslint.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -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() {
Expand Down Expand Up @@ -219,5 +218,4 @@ module.exports = defineConfig([
...tsConfigs,
testConfig,
mdxConfig,
...storybookPlugin.configs['flat/recommended'],
]);
1 change: 0 additions & 1 deletion tools/scripts-config-eslint/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -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"
Expand Down
12 changes: 1 addition & 11 deletions tools/scripts-core/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -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": {
Expand All @@ -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",
Expand Down
3 changes: 0 additions & 3 deletions tools/scripts-core/src/config/webpack.config.js
Original file line number Diff line number Diff line change
@@ -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';
Expand All @@ -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 }),
Expand Down
40 changes: 0 additions & 40 deletions tools/scripts-core/src/index.js
Original file line number Diff line number Diff line change
@@ -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';
Expand All @@ -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);
}
Expand Down Expand Up @@ -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);
Expand Down
156 changes: 0 additions & 156 deletions tools/scripts-core/src/scripts/lint-merge-report.js

This file was deleted.

Loading
Loading