Skip to content

[DependencyInjection] Replace link to getConfiguration method #14759

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
Dec 30, 2020
Merged

[DependencyInjection] Replace link to getConfiguration method #14759

merged 1 commit into from
Dec 30, 2020

Conversation

sebpacz
Copy link
Contributor

@sebpacz sebpacz commented Dec 30, 2020 •
edited
Loading

In my opinion, the link should redirect to the class where we can preview the method that must be overridden.

@carsonbot carsonbot added this to the 4.4 milestone Dec 30, 2020
@sebpacz @spaczkowski
Replace link to getConfiguration method
6fa2894 
In my opinion, the link should redirect to the class where we can preview the method that must be overridden.
@carsonbot carsonbot changed the title Replace link to getConfiguration method [DependencyInjection] Replace link to getConfiguration method Dec 30, 2020
@javiereguiluz
Copy link
Member

Thanks Sebastian.

@javiereguiluz javiereguiluz merged commit 52a40b3 into symfony:4.4 Dec 30, 2020
@sebpacz sebpacz deleted the patch-7 branch December 30, 2020 20:26
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
DependencyInjection Status: Reviewed
Projects
None yet
Milestone
4.4
Development

Successfully merging this pull request may close these issues.
3 participants
@sebpacz @javiereguiluz @carsonbot