Skip to content

feat!: remove all production deps#221

Draft
erickzhao wants to merge 3 commits intomainfrom
remove-production-deps
Draft

feat!: remove all production deps#221
erickzhao wants to merge 3 commits intomainfrom
remove-production-deps

Conversation

@erickzhao
Copy link
Copy Markdown
Member

@erickzhao erickzhao commented Apr 2, 2026

BREAKING CHANGE: updateInterval must specify a number in milliseconds rather than a string.

Two packages are dropped here:

  • Removing ms in favour of just passing milliseconds in (might be fruitless in the grand scheme of a project's node_modules tree unless chore: inlining ms debug-js/debug#1018 lands).
  • Removing github-url-to-object in favour of just a regex. It's AI-generated, but tests were added to validate the input/output.

@erickzhao erickzhao requested a review from a team as a code owner April 2, 2026 00:36
@erickzhao erickzhao marked this pull request as draft April 2, 2026 01:23
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.

2 participants