-
Notifications
You must be signed in to change notification settings - Fork 10.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: Add reset
script
#10627
build: Add reset
script
#10627
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of maintaining a list of purgable files/folder in every package, could we use git clean -xd
instead (with --exclude
to skip deleting certain files/folders)?
We often face situations where a build will fail because some cache somewhere is messing up with the build. This PR adds a `nuke` script that can be used to delete all files that might be out of date (i.e. node_modules, dist files, .turbo cache, tsbuildinfo files, etc). It is a more destructive version from `clean`, which doesn't touch e.g. node_modules.
This reverts commit 67b28bb.
This reverts commit 8661a8f.
67b28bb
to
8ea8f77
Compare
@netroy I updated the script to be a bit more sophisticated. Added |
Co-authored-by: कारतोफ्फेलस्क्रिप्ट™ <netroy@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this should work, but has one minor issue that if someone runs pnpm reset
before running pnpm i
, this script will error out.
Fixed this by ensuring zx is available. Not necessary ideal to run |
✅ All Cypress E2E specs passed |
n8n Run #6719
Run Properties:
|
Project |
n8n
|
Branch Review |
add-nuke-script
|
Run status |
Passed #6719
|
Run duration | 04m 43s |
Commit |
3118a03d0b: 🌳 🖥️ browsers:node18.12.0-chrome107 🤖 tomi 🗃️ e2e/*
|
Committer | Tomi Turtiainen |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
1
|
Pending |
0
|
Skipped |
0
|
Passing |
423
|
View all changes introduced in this branch ↗︎ |
* master: refactor(RabbitMQ Trigger Node): Improve type-safety, add tests, and fix issues with manual triggers (#10663) feat(editor): Add support for nodes with multiple main inputs in new canvas (no-changelog) (#10659) fix(editor): Set minimum zoom to 0 to allow fitting very large workflows in new canvas (no-changelog) (#10666) feat(editor): Change selection to be default canvas behaviour (no-changelog) (#10668) feat: More hints to nodes (#10565) fix(editor): Fix opening executions tab from a new, unsaved workflow (#10652) fix(AI Agent Node): Fix tools agent when using memory and Anthropic models (#10513) feat(editor): Make highlighted data pane floating (#10638) fix(editor): Fix workflow loading after switching to executions view in new canvas (no-changelog) (#10655) refactor(benchmark): Separate cloud env provisioning from running benchmarks (#10657) feat(core): Implement wrapping of regular nodes as AI Tools (#10641) refactor(editor): Remove Trial logic in personalization modal and port to script setup (#10649) fix(core): Declutter webhook insertion errors (#10650) feat: Reintroduce collaboration feature (#10602) feat(benchmark): Add scenario for expressions with Set node (#10647) feat(benchmark): Add benchmark scenario for binary files (#10648) build: Add `reset` script (#10627) feat(editor): Overhaul node insert position computation in new canvas (no-changelog) (#10637)
Got released with |
Summary
We often face situations where a build will fail because some cache somewhere is messing up with the build. This PR adds a
nuke
script that can be used to delete all files that might be out of date (i.e. node_modules, dist files, .turbo cache, tsbuildinfo files, etc). It is a more destructive version fromclean
, which doesn't touch e.g. node_modules.Review / Merge checklist
release/backport
(if the PR is an urgent fix that needs to be backported)