Skip to content

Add modern-di to Third-Party Extensions#17

Merged
sobolevn merged 2 commits into
litestar-org:mainfrom
lesnik512:add-modern-di
Jun 25, 2026
Merged

Add modern-di to Third-Party Extensions#17
sobolevn merged 2 commits into
litestar-org:mainfrom
lesnik512:add-modern-di

Conversation

@lesnik512

Copy link
Copy Markdown
Contributor

Adds modern-di (≈56★, MIT) to Third-Party Extensions → General: a DI framework with IoC container and scopes, with a dedicated Litestar integration. Single item with a description, placed alongside the existing dishka entry.

@lesnik512
lesnik512 marked this pull request as ready for review June 20, 2026 18:39
@lesnik512
lesnik512 requested review from a team as code owners June 20, 2026 18:39

@sobolevn sobolevn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you please start with fixing the CI? So, we can rebase your PRs and review / merge them after that.

The link checker was failing on pre-existing dead/redirecting links
unrelated to the modern-di addition (main has been red on these too):

- OpenTelemetry docs moved to reference/plugins/opentelemetry.html
- litestar-saq, litestar-svcs, litestar-users repos transferred owners
- railway.app -> railway.com; drop two dead Railway deploy-template
  buttons (templates now 404)
- drop litestar-dishka-faststream entry (repo migrated to FastAPI)
- whitelist bot-blocked blog.litestar.dev (403) in CI link check
@lesnik512

lesnik512 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor Author

The awesome-bot link check was already red on main (unrelated to this PR — my modern-di links pass). I included a commit fixing the pre-existing stale links so CI can go green:

  • OpenTelemetry docs → moved to reference/plugins/opentelemetry.html (was 404)
  • litestar-saq / litestar-svcs / litestar-users → updated to the repos' new owners (301 redirects)
  • railway.app → railway.com, and dropped two Railway deploy-template buttons whose templates now 404
  • litestar-dishka-faststream → removed: the repo was renamed and migrated to FastAPI (faststream-community/fastapi-dishka-faststream), so it no longer fits an awesome-Litestar list
  • blog.litestar.dev → whitelisted in the CI link check (returns 403 to the bot but is fine for humans)

Verified locally: awesome_bot README.md --allow 302,307,429 --allow-ssl --allow-dupe --white-list ... reports No issues.

@lesnik512
lesnik512 requested a review from sobolevn June 25, 2026 19:11

@sobolevn sobolevn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@sobolevn
sobolevn merged commit c270473 into litestar-org:main Jun 25, 2026
@lesnik512
lesnik512 deleted the add-modern-di branch June 25, 2026 19:29
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