mirror of
https://github.com/n8n-io/n8n.git
synced 2025-01-11 12:57:29 -08:00
ci: Remove eslint diff plugin. Run full linting on PRs (no-changelog) (#7175)
This commit is contained in:
parent
54bf66d335
commit
c18ba370d5
7
.github/workflows/ci-pull-requests.yml
vendored
7
.github/workflows/ci-pull-requests.yml
vendored
|
@ -67,10 +67,5 @@ jobs:
|
|||
path: ./packages/**/dist
|
||||
key: ${{ github.sha }}-base:18-test-lint
|
||||
|
||||
- name: Fetch base branch for `git diff`
|
||||
run: git fetch origin ${{ github.event.pull_request.base.ref }}:${{ github.event.pull_request.base.ref }}
|
||||
|
||||
- name: Run ESLint on changes only
|
||||
env:
|
||||
ESLINT_PLUGIN_DIFF_COMMIT: ${{ github.event.pull_request.base.ref }}
|
||||
- name: Lint
|
||||
run: pnpm lint
|
||||
|
|
|
@ -491,19 +491,3 @@ const config = (module.exports = {
|
|||
},
|
||||
],
|
||||
});
|
||||
|
||||
if ('ESLINT_PLUGIN_DIFF_COMMIT' in process.env) {
|
||||
/**
|
||||
* Plugin to lint only changes
|
||||
*
|
||||
* https://github.com/paleite/eslint-plugin-diff#plugindiffdiff-recommended
|
||||
*/
|
||||
config.plugins.push('eslint-plugin-diff');
|
||||
|
||||
/**
|
||||
* Config for eslint-plugin-diff
|
||||
*
|
||||
* https://github.com/paleite/eslint-plugin-diff#plugindiffdiff-recommended
|
||||
*/
|
||||
config.extends.push('plugin:diff/diff');
|
||||
}
|
||||
|
|
|
@ -11,7 +11,6 @@
|
|||
"eslint-config-airbnb-typescript": "^17.1.0",
|
||||
"eslint-config-prettier": "^8.9.0",
|
||||
"eslint-import-resolver-typescript": "^3.5.5",
|
||||
"eslint-plugin-diff": "^2.0.1",
|
||||
"eslint-plugin-import": "^2.28.0",
|
||||
"eslint-plugin-n8n-local-rules": "^1.0.0",
|
||||
"eslint-plugin-prettier": "^5.0.0",
|
||||
|
|
|
@ -3,7 +3,6 @@ import {
|
|||
MAPPING_COLUMNS_RESPONSE,
|
||||
NODE_PARAMETER_VALUES,
|
||||
UPDATED_SCHEMA,
|
||||
getLatestValueChangeEvent,
|
||||
} from './utils/ResourceMapper.utils';
|
||||
import { useNodeTypesStore } from '@/stores/nodeTypes.store';
|
||||
import { waitAllPromises } from '@/__tests__/utils';
|
||||
|
|
|
@ -167,9 +167,6 @@ importers:
|
|||
eslint-import-resolver-typescript:
|
||||
specifier: ^3.5.5
|
||||
version: 3.5.5(@typescript-eslint/parser@6.2.0)(eslint-plugin-import@2.28.0)(eslint@8.45.0)
|
||||
eslint-plugin-diff:
|
||||
specifier: ^2.0.1
|
||||
version: 2.0.1(eslint@8.45.0)
|
||||
eslint-plugin-import:
|
||||
specifier: ^2.28.0
|
||||
version: 2.28.0(@typescript-eslint/parser@6.2.0)(eslint-import-resolver-typescript@3.5.5)(eslint@8.45.0)
|
||||
|
@ -6765,7 +6762,7 @@ packages:
|
|||
ts-dedent: 2.2.0
|
||||
type-fest: 3.13.1
|
||||
vue: 3.3.4
|
||||
vue-component-type-helpers: 1.8.10
|
||||
vue-component-type-helpers: 1.8.11
|
||||
transitivePeerDependencies:
|
||||
- encoding
|
||||
- supports-color
|
||||
|
@ -11750,15 +11747,6 @@ packages:
|
|||
- supports-color
|
||||
dev: true
|
||||
|
||||
/eslint-plugin-diff@2.0.1(eslint@8.45.0):
|
||||
resolution: {integrity: sha512-qqbvwaaO1cfkUprliqiRojRsD0qGsvzmJNqNrb9s0h15sDVzZMXYdu0TUFpUwauLeU28etSsfWIp0Uu+OAcXXw==}
|
||||
engines: {node: '>=14.0.0'}
|
||||
peerDependencies:
|
||||
eslint: '>=6.7.0'
|
||||
dependencies:
|
||||
eslint: 8.45.0
|
||||
dev: true
|
||||
|
||||
/eslint-plugin-import@2.28.0(@typescript-eslint/parser@6.2.0)(eslint-import-resolver-typescript@3.5.5)(eslint@8.45.0):
|
||||
resolution: {integrity: sha512-B8s/n+ZluN7sxj9eUf7/pRFERX0r5bnFA2dCaLHy2ZeaQEAz0k+ZZkFWRFHJAqxfxQDx6KLv9LeIki7cFdwW+Q==}
|
||||
engines: {node: '>=4'}
|
||||
|
@ -21744,8 +21732,8 @@ packages:
|
|||
vue: 3.3.4
|
||||
dev: false
|
||||
|
||||
/vue-component-type-helpers@1.8.10:
|
||||
resolution: {integrity: sha512-FJtmfw2Gn6eQ8kAVNEhw9nYIzWmVQJjdyQRtJXZ7tgXh/FoZhQnZ2KyxR+NuF9U4iZLBvSspeetIpnP9yxxyMw==}
|
||||
/vue-component-type-helpers@1.8.11:
|
||||
resolution: {integrity: sha512-CWItFzuEWjkSXDeMGwQEc5cFH4FaueyPQHfi1mBDe+wA2JABqNjFxFUtmZJ9WHkb0HpEwqgBg1umiXrWYXkXHw==}
|
||||
dev: true
|
||||
|
||||
/vue-component-type-helpers@1.8.4:
|
||||
|
|
Loading…
Reference in a new issue