Skip to content

Navigation Menu

Sign in

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

[DoctrineBridge] Unused namespaces #60276

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[DoctrineBridge] Unused namespaces #60276

wants to merge 1 commit into from

Conversation

wkania
Copy link
Contributor

@wkania wkania commented Apr 25, 2025 •
edited
Loading

Q A
Branch? 7.2
Bug fix? yes
New feature? no
Deprecations? no
Issues
License MIT

Test failed in the PR.
This one will be merged probably faster.

@carsonbot carsonbot added this to the 7.2 milestone Apr 25, 2025
@wkania wkania force-pushed the unused-namespace-and-undefined-variable branch from d9d5102 to 44f0fcc Compare April 26, 2025 17:14
@wkania wkania changed the title [DoctrineBridge] unused namespaces and undefined variable [DoctrineBridge] Unused namespaces Apr 26, 2025
@fabpot fabpot mentioned this pull request Apr 27, 2025
Merged
@fabpot
Copy link
Member

fabpot commented Apr 27, 2025

Good catch, I've done in on the entire code base (and on 6.4) in #60283.

@fabpot fabpot closed this Apr 27, 2025
fabpot added a commit that referenced this pull request Apr 27, 2025
@fabpot
minor #60283 Remove unneeded use statements (fabpot)
bfe2d66 
This PR was merged into the 6.4 branch.

Discussion
----------

Remove unneeded use statements

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Issues        | Fix #60276
| License       | MIT

Commits
-------

88f6907 Remove unneeded use statements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
DoctrineBridge Status: Needs Review
Projects
None yet
Milestone
7.2
Development

Successfully merging this pull request may close these issues.
3 participants
@wkania @fabpot @carsonbot
Morty Proxy This is a proxified and sanitized view of the page, visit original site.