Skip to content
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

[11.x] Remove unused namespace #51436

Merged
merged 2 commits into from
May 17, 2024
Merged

[11.x] Remove unused namespace #51436

merged 2 commits into from
May 17, 2024

Conversation

saMahmoudzadeh
Copy link
Contributor

@saMahmoudzadeh saMahmoudzadeh commented May 17, 2024 •
edited
Loading

This PR, fixes namespace ( remove unused namespace Builder.php , Redis.php, InteractsWithExceptionHandling.php)

@driesvints driesvints merged commit feaef1a into laravel:11.x May 17, 2024
28 checks passed
@driesvints
Copy link
Member

thanks!

@saMahmoudzadeh saMahmoudzadeh deleted the fix/namespace branch May 17, 2024 07:20
@vlakoff
Copy link
Contributor

vlakoff commented May 19, 2024 •
edited
Loading

We don't have implemented some GitHub action for automated CS checks, for instance here, automatic detection of unused namespaces?

@renovate renovate bot mentioned this pull request Jun 27, 2024
Merged
1 task
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@saMahmoudzadeh @driesvints @vlakoff