Skip to content

chore: remove commented-out import_from_string utility in _legacy.py - #6932

Open
detail-app[bot] wants to merge 1 commit into
mainfrom
detail/dead-code/chore-remove-commented-out-import-from-string-util-57c869
Open

chore: remove commented-out import_from_string utility in _legacy.py#6932
detail-app[bot] wants to merge 1 commit into
mainfrom
detail/dead-code/chore-remove-commented-out-import-from-string-util-57c869

Conversation

@detail-app

@detail-app detail-app Bot commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Remove the commented-out import_from_string helper (27 lines, plus
its uvicorn source-URL header) from cli/_legacy.py. It was copied as
a commented reference from uvicorn's importer and has had zero callers
anywhere in the codebase since it was introduced — the module:attribute
auto-discovery mechanism it would have supported was superseded by the
discover module (used directly by __main__.py).

History


Dead Code PRs can be configured here.

@detail-app
detail-app Bot requested a review from a team as a code owner August 20, 2026 21:08
@detail-app
detail-app Bot requested a review from theomonnom August 20, 2026 21:08

@devin-ai-integration devin-ai-integration Bot 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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.

1 participant