8000 Fix compiler pass phase when extensions that implements CompilerPassI… by l-vo · Pull Request #12986 · symfony/symfony-docs · GitHub
[go: up one dir, main page]
Skip to content

Fix compiler pass phase when extensions that implements CompilerPassI… #12986

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

Fix compiler pass phase when extensions that implements CompilerPassI… #12986

wants to merge 1 commit into from

Conversation

l-vo
Copy link
Contributor
@l-vo l-vo commented Jan 25, 2020

Fix compiler pass phase when extensions that implements CompilerPassnterface are executed

HeahDude
HeahDude approved these changes Feb 4, 2020
View reviewed changes
Copy link
Contributor
@HeahDude HeahDude left a comment

Choose a reason for hiding this comment

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

👍

@HeahDude HeahDude added this to the 3.4 milestone Feb 4, 2020
@OskarStark
Copy link
Contributor

Thank you @l-vo.

OskarStark added a commit that referenced this pull request Feb 5, 2020
@OskarStark
minor #12986 Fix compiler pass phase when extensions that implements …
4288b0f 
…CompilerPassI… (l-vo)

This PR was merged into the 3.4 branch.

Discussion
----------

Fix compiler pass phase when extensions that implements CompilerPassI…

Fix compiler pass phase when extensions that implements CompilerPassnterface are executed

Commits
-------

0a255dc Fix compiler pass phase when extensions that implements CompilerPassInterface are executed
@l-vo l-vo closed this Feb 5, 2020
@l-vo l-vo deleted the fix_compilerpass_extension_compile_phase branch February 5, 2020 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@OskarStark OskarStark OskarStark approved these changes

@HeahDude HeahDude HeahDude approved these changes

Assignees
No one assigned
Labels
DependencyInjection Status: Reviewed
Projects
None yet
Milestone
3.4
Development

Successfully merging this pull request may close these issues.
4 participants
@l-vo @OskarStark @HeahDude @carsonbot
0