mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-03-05 20:50:17 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

ci: Add checklist item hinting at not adding new dependencies for nodes (no-changelog) (#5985)

Browse source 
* ci: Add checklist item hinting at not adding new dependencies for nodes

* Update CHECKLIST.yml

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>

---------

Co-authored-by: Iván Ovejero <ivov.src@gmail.com>
This commit is contained in:
Cornelius Suermann 2023-04-16 16:51:48 +02:00 committed by GitHub
parent 4c994faec1
commit 026d0bc909
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 19 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
CHECKLIST.yml
View file

@ -1,46 +1,48 @@
paths: paths:
"packages/**": 'packages/**':
- If fixing bug, added test to cover scenario. - If fixing bug, added test to cover scenario.
- If addressing forum or Github issue, added link to description. - If addressing forum or Github issue, added link to description.
"packages/**/*.ts": 'packages/**/*.ts':
- Added unit tests to cover new or updated functionality. - Added unit tests to cover new or updated functionality.
"**/*.vue": '**/*.vue':
- Used composition API for all new components. - Used composition API for all new components.
- Added component or unit tests to cover functionality. - Added component or unit tests to cover functionality.
# cli # cli
"packages/cli/src/databases/migrations/**": 'packages/cli/src/databases/migrations/**':
- Requested review from at least two engineers on migration. - Requested review from at least two engineers on migration.
- Avoided irreversible data migrations. - Avoided irreversible data migrations.
- Avoided deleting or updating data keys. - Avoided deleting or updating data keys.
- Wrote 'down' migration if possible. - Wrote 'down' migration if possible.
"n8n/packages/cli/src/api/**": 'n8n/packages/cli/src/api/**':
- Added integration tests for new endpoints. - Added integration tests for new endpoints.
# editor ui # editor ui
"packages/editor-ui/**/*.vue": 'packages/editor-ui/**/*.vue':
- Added E2E if adding new features. - Added E2E if adding new features.
- Used design system tokens (colors, spacings...) where possible. - Used design system tokens (colors, spacings...) where possible.
"packages/editor-ui/src/mixins/restApi.ts": 'packages/editor-ui/src/mixins/restApi.ts':
- Avoided adding new methods. Only deleted from here. - Avoided adding new methods. Only deleted from here.
"packages/editor-ui/src/mixins/**": 'packages/editor-ui/src/mixins/**':
- Avoided adding new mixins (use composables instead). Only removed code from here. - Avoided adding new mixins (use composables instead). Only removed code from here.
"packages/editor-ui/src/views/NodeView.vue": 'packages/editor-ui/src/views/NodeView.vue':
- Avoided adding code here. Only refactored to make it smaller. - Avoided adding code here. Only refactored to make it smaller.
"packages/editor-ui/src/hooks/**": 'packages/editor-ui/src/hooks/**':
- Avoided adding new hooks. Only refactored to move hooks to relevant store instead. - Avoided adding new hooks. Only refactored to move hooks to relevant store instead.
# nodes-base # nodes-base
"packages/nodes-base/nodes/**": 'packages/nodes-base/nodes/**':
- Added workflow tests for nodes if possible. - Added workflow tests for nodes if possible.
'packages/nodes-base/package.json':
- Avoided adding dependencies for nodes if not absolutely necessary.
# design-system # design-system
"packages/design-system/**/*.vue": 'packages/design-system/**/*.vue':
- Used design system tokens (colors, spacings...) where possible. - Used design system tokens (colors, spacings...) where possible.
- Updated Storybook with new component or updated functionality. - Updated Storybook with new component or updated functionality.
# e2e # e2e
"cypress/e2e/**": 'cypress/e2e/**':
- Avoided chaining commands more than two or three times (to avoid flakiness because only last one will be retried). - Avoided chaining commands more than two or three times (to avoid flakiness because only last one will be retried).
- Spoofed endpoints that are not critical for the test (to avoid flakiness). - Spoofed endpoints that are not critical for the test (to avoid flakiness).
- Picked most efficient path to start the test (for example skipped account setup and starting at /workflow/new for a canvas test). - Picked most efficient path to start the test (for example skipped account setup and starting at /workflow/new for a canvas test).