Skip to content

chore: add a clean script for maintainers#2583

Merged
WilliamBergamin merged 2 commits intomainfrom
clean-script
May 6, 2026
Merged

chore: add a clean script for maintainers#2583
WilliamBergamin merged 2 commits intomainfrom
clean-script

Conversation

@WilliamBergamin
Copy link
Copy Markdown
Contributor

@WilliamBergamin WilliamBergamin commented May 6, 2026

Summary

These changes add a clean script for the project to clean up dependencies and package lock files, this allows for easier local development

  • git clean -xdf node_modules packages/**/node_modules — recursively removes all node_modules directories (ignored or untracked), including the root one. The -d flag removes directories, -x includes gitignored files, -f forces.
  • git clean -xf '**/package-lock.json' — removes any package-lock.json that is NOT tracked by git. Git-tracked files are never touched by git clean, so the root lock file stays.

Requirements

@WilliamBergamin WilliamBergamin self-assigned this May 6, 2026
@WilliamBergamin WilliamBergamin requested a review from a team as a code owner May 6, 2026 16:35
@WilliamBergamin WilliamBergamin added the tests M-T: Testing work only label May 6, 2026
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 6, 2026

⚠️ No Changeset found

Latest commit: 1d2b938

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Copy Markdown
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@WilliamBergamin Thanks for thinking about this!

I'm wondering if we want to also remove the "dist" directory as part of this to avoid strange publications in odd CI environments? No blocker! 🚢 💨

@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.50%. Comparing base (5bc7685) to head (1d2b938).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2583   +/-   ##
=======================================
  Coverage   87.50%   87.50%           
=======================================
  Files          62       62           
  Lines       10256    10256           
  Branches      418      418           
=======================================
  Hits         8974     8974           
  Misses       1260     1260           
  Partials       22       22           
Flag Coverage Δ
cli-hooks 87.50% <ø> (ø)
cli-test 87.50% <ø> (ø)
logger 87.50% <ø> (ø)
oauth 87.50% <ø> (ø)
socket-mode 87.50% <ø> (ø)
web-api 87.50% <ø> (ø)
webhook 87.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@WilliamBergamin WilliamBergamin enabled auto-merge (squash) May 6, 2026 17:16
@WilliamBergamin WilliamBergamin merged commit e8a087c into main May 6, 2026
12 checks passed
@WilliamBergamin WilliamBergamin deleted the clean-script branch May 6, 2026 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests M-T: Testing work only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants