Skip to content

fix(deps): 🐛 Bundle core DI dependencies#8

Merged
haskou merged 1 commit into
mainfrom
fix/core-di-dependencies
Jun 25, 2026
Merged

fix(deps): 🐛 Bundle core DI dependencies#8
haskou merged 1 commit into
mainfrom
fix/core-di-dependencies

Conversation

@haskou

@haskou haskou commented Jun 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • Moves core DI runtime packages node-dependency-injection and fs-extra into package dependencies.
  • Removes those packages from optional peers so applications do not need to install them explicitly.
  • Declares class-transformer and class-validator as optional Express/routing-controllers peers and updates installation docs.

Validation

  • yarn format:check
  • yarn lint
  • yarn typecheck
  • yarn test:coverage (100%)
  • yarn docs:build
  • yarn --cwd example typecheck
  • yarn build
  • npx -y depcheck --json

@codecov-commenter

codecov-commenter commented Jun 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@haskou haskou merged commit 32cb6ce into main Jun 25, 2026
2 checks passed
@haskou haskou deleted the fix/core-di-dependencies branch June 25, 2026 20:19
github-actions Bot added a commit that referenced this pull request Jun 25, 2026
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