Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fabpot merged 1 commit intosymfony:masterfromgarak:remove-getprojectdir
May 6, 2020

Conversation

@garak
Copy link
Contributor

QA
Branch?master
Bug fix?yes
New feature?no
Deprecations?no
TicketsFix#36719
LicenseMIT
Doc PRnot 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
ContributorAuthor

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

@garakgarakforce-pushed theremove-getprojectdir branch from3696245 to51697a3CompareMay 6, 2020 09:46
@garak
Copy link
ContributorAuthor

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

Ah, I see.
Changelog updated!

@nicolas-grekasnicolas-grekas changed the titleremove getProjectDir method from MicroKernelTrait[FrameworkBundle] remove getProjectDir method from MicroKernelTraitMay 6, 2020
@garakgarakforce-pushed theremove-getprojectdir branch from1d51402 tof2f3ebaCompareMay 6, 2020 10:44
@fabpot
Copy link
Member

Thank you@garak.

garak reacted with thumbs up emoji

@fabpotfabpot merged commit75e71e3 intosymfony:masterMay 6, 2020
@fabpotfabpot mentioned this pull requestMay 16, 2020
@garakgarak deleted the remove-getprojectdir branchDecember 15, 2020 16:15
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@fabpotfabpotfabpot approved these changes

@nicolas-grekasnicolas-grekasnicolas-grekas approved these changes

@xabbuhxabbuhxabbuh approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

5.1

Development

Successfully merging this pull request may close these issues.

New getProjectDir in MicroKernelTrait is too opinionated

5 participants

@garak@nicolas-grekas@fabpot@xabbuh@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp