Skip to content

[Notifier] Remove deprecated TransportFactoryTestCase #60990

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
Merged
merged 1 commit into from
Jul 2, 2025
Merged

[Notifier] Remove deprecated TransportFactoryTestCase #60990

merged 1 commit into from
Jul 2, 2025
+8 −49

Conversation

mttsch
Copy link
Contributor

@mttsch mttsch commented Jun 30, 2025

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

@mttsch mttsch requested a review from OskarStark as a code owner June 30, 2025 18:05
@carsonbot carsonbot added this to the 8.0 milestone Jun 30, 2025
@mttsch
Copy link
Contributor Author

mttsch commented Jun 30, 2025

Symfony docs do not need update because removed class is not referenced, also see symfony/symfony-docs#20265.

@mttsch mttsch force-pushed the feature/notifier_TransportFactoryTestCase branch from e404103 to 2384530 Compare June 30, 2025 18:25
@mttsch mttsch changed the title [Notifier] Remove deprecated TransportFactoryTestCase [Notifier] Remove deprecated TransportFactoryTestCase Jun 30, 2025
OskarStark
OskarStark reviewed Jul 1, 2025
View reviewed changes
@mttsch mttsch force-pushed the feature/notifier_TransportFactoryTestCase branch from 2384530 to 7866b17 Compare July 2, 2025 03:15
alexandre-daubois
alexandre-daubois reviewed Jul 2, 2025
View reviewed changes
@fabpot
Copy link
Member

fabpot commented Jul 2, 2025

Thank you @mttsch.

@fabpot fabpot merged commit 7dfef9e into symfony:8.0 Jul 2, 2025
8 of 9 checks passed
@fabpot fabpot mentioned this pull request Jul 2, 2025
Closed
@mttsch mttsch deleted the feature/notifier_TransportFactoryTestCase branch July 2, 2025 19:10
@welcoMattic welcoMattic added the BC Layer removal Used to track BC layer removals before a major release label Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabpot fabpot fabpot approved these changes

@OskarStark OskarStark OskarStark approved these changes

@xabbuh xabbuh xabbuh approved these changes

@alexandre-daubois alexandre-daubois alexandre-daubois approved these changes

Assignees
No one assigned
Labels
BC Layer removal Used to track BC layer removals before a major release Feature Notifier Status: Reviewed
Projects
None yet
Milestone
8.0
Development

Successfully merging this pull request may close these issues.
7 participants
@mttsch @fabpot @OskarStark @xabbuh @alexandre-daubois @welcoMattic @carsonbot