Skip to content

chore: cleanup some old unused code#7856

Merged
martinjagodic merged 11 commits into
decaporg:mainfrom
delucis:chris/cleanup
Jun 12, 2026
Merged

chore: cleanup some old unused code#7856
martinjagodic merged 11 commits into
decaporg:mainfrom
delucis:chris/cleanup

Conversation

@delucis

@delucis delucis commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Summary

Spent a little bit of time investigating superfluous stuff in the repo using Knip. There’s quite a lot of potential cleanup work, but this PR sticks to some low hanging fruit (also to not make the PR too big to review).

  • Removes some unused dependencies (mostly related to Webpack etc., likely left over from previous configurations)
  • Refactors one small bit of code to use new URL() so the url npm package is no longer needed
  • Deletes some code files that aren’t used anywhere (not imported by anything, not exposed publicly)

There’s more work where this is coming from, but next step would be removing unused exports etc. from code, which is a bit more delicate, so will save that for future PRs.

Test plan

Existing builds and tests should pass

Checklist

Please add a x inside each checkbox:

A picture of a cute animal (not mandatory but encouraged)

@delucis delucis requested a review from a team as a code owner June 11, 2026 18:00

@yanthomasdev yanthomasdev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Amazing work, LGTM!

@martinjagodic martinjagodic merged commit 8d83ac2 into decaporg:main Jun 12, 2026
6 checks passed
@delucis delucis deleted the chris/cleanup branch June 14, 2026 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants