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.

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

[FrameworkBundle] Deprecate creating public workflow services #40875

Merged

Conversation

fancyweb
Copy link
Contributor

@fancyweb fancyweb commented Apr 19, 2021

Q A
Branch? 5.x
Bug fix? no
New feature? no
Deprecations? yes
Tickets -
License MIT
Doc PR -

AFAIK, workflow services don't need to be public. friendly ping to @lyrixx as a reviewer.

UPGRADE-5.3.md Outdated Show resolved Hide resolved
UPGRADE-5.3.md Outdated Show resolved Hide resolved
@fancyweb fancyweb force-pushed the fwb/deprecate-workflow-abstract-public branch from b22b383 to e6f599a Compare April 20, 2021 07:19
UPGRADE-6.0.md Outdated Show resolved Hide resolved
Copy link
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

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

(once the duplicate lines in the changelogs are removed)

@fancyweb fancyweb force-pushed the fwb/deprecate-workflow-abstract-public branch from e6f599a to 79cbf00 Compare April 20, 2021 11:18
@nicolas-grekas nicolas-grekas added this to the 5.3 milestone Apr 20, 2021
@nicolas-grekas
Copy link
Member

Thank you @fancyweb.

@nicolas-grekas nicolas-grekas merged commit be384cf into symfony:5.x Apr 21, 2021
@fancyweb fancyweb deleted the fwb/deprecate-workflow-abstract-public branch April 21, 2021 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants