Skip to content

chore(many): remove unused dependencies#2536

Merged
HerrTopi merged 1 commit into
masterfrom
remove-dependencies
May 7, 2026
Merged

chore(many): remove unused dependencies#2536
HerrTopi merged 1 commit into
masterfrom
remove-dependencies

Conversation

@HerrTopi

@HerrTopi HerrTopi commented May 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Drop dependency declarations that are not imported anywhere in source:

  • docs-app: buffer, marked, semver (marked-react pulls marked transitively; no import sites for buffer/semver)
  • ui-date-input: moment-timezone (the Moment type flows in via @instructure/ui-i18n; nothing imports moment-timezone directly)
  • ui-scripts: mocha (residual from the legacy ui-test --mocha flow; vitest is the test runner)

Test plan

  • pnpm install resolves cleanly
  • pnpm run build:types succeeds
  • pnpm run test:vitest passes
  • Docs app builds and runs (pnpm run dev)

@github-actions

github-actions Bot commented May 5, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-05-07 13:49 UTC

@HerrTopi HerrTopi requested review from balzss and matyasf and removed request for matyasf May 5, 2026 14:15
@HerrTopi HerrTopi force-pushed the remove-dependencies branch from 2d6383d to d395b18 Compare May 7, 2026 11:21
@github-actions

github-actions Bot commented May 7, 2026

Copy link
Copy Markdown

Visual regression report

No changes.

Status Count
Unchanged 32
Changed 0
New 0
Removed 0

📊 View full report

Baselines come from the visual-baselines branch. They refresh on every merge to master.

github-actions Bot pushed a commit that referenced this pull request May 7, 2026
…) 0a0b2ac9304d922795e690e0d67d7d67b5025211
@HerrTopi HerrTopi merged commit 88f3c40 into master May 7, 2026
8 checks passed
@HerrTopi HerrTopi deleted the remove-dependencies branch May 7, 2026 13:48
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