Skip to content

[FrameworkBundle] remove getProjectDir method from MicroKernelTrait #36721

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

Merged
merged 1 commit into from
May 6, 2020

Conversation

garak
Copy link
Contributor

@garak garak commented May 6, 2020

Q A
Branch? master
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #36719
License MIT
Doc PR not needed

Remove method added in trait, to be able to use same method in base kernel class.

@nicolas-grekas
Copy link
Member

Please fix the changelog also.

@garak
Copy link
Contributor Author

garak commented May 6, 2020

Please fix the changelog also.

I thought changelogs were autogenerated...

@nicolas-grekas
Copy link
Member

Only the main changelog is. We maintain them per components.
Check the commit you linked in the issue:
a689807

@garak garak force-pushed the remove-getprojectdir branch from 3696245 to 51697a3 Compare May 6, 2020 09:46
@garak
Copy link
Contributor Author

garak commented May 6, 2020

Only the main changelog is. We maintain them per components.

Ah, I see.
Changelog updated!

@nicolas-grekas nicolas-grekas changed the title remove getProjectDir method from MicroKernelTrait [FrameworkBundle] remove getProjectDir method from MicroKernelTrait May 6, 2020
@garak garak force-pushed the remove-getprojectdir branch from 1d51402 to f2f3eba Compare May 6, 2020 10:44
@fabpot
Copy link
Member

fabpot commented May 6, 2020

Thank you @garak.

@fabpot fabpot merged commit 75e71e3 into symfony:master May 6, 2020
@fabpot fabpot mentioned this pull request May 16, 2020
@garak garak deleted the remove-getprojectdir branch December 15, 2020 16:15
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.

New getProjectDir in MicroKernelTrait is too opinionated
5 participants