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

map psr4 classes into Bundle subfolder #5

Merged
merged 1 commit into from Feb 26, 2018

Conversation

Projects
None yet
4 participants
@kaufmo
Copy link
Contributor

kaufmo commented Feb 22, 2018

No description provided.

@kaufmo kaufmo requested review from nsams and fraxachun Feb 22, 2018

@nsams

This comment has been minimized.

Copy link
Member

nsams commented Feb 22, 2018

bitte um abstimmung bin benjamin damit die bundles gleich strukturiert sind https://gitlab.vivid-planet.com/vivid/kwf-carlog-auth

@nsams nsams requested a review from Ben-Ho Feb 22, 2018

@kaufmo

This comment has been minimized.

Copy link
Contributor Author

kaufmo commented Feb 23, 2018

bissl a unterschied ist da schon, das eine ist nur ein Bundle da macht der zwischenordner keinen Sinn und hier haben wir eben koala komponenten auch dabei da macht er wiederum schon sinn, die klassen würden durch diese einstellung jedoch bei beiden die gleiche struktur haben ;)

@nsams

nsams approved these changes Feb 23, 2018

Copy link
Member

nsams left a comment

ok.... benjamin soll trotzdem sein OK dafür geben weil der hat sich damit schon recht auseinander gesetzt...

@Ben-Ho

This comment has been minimized.

Copy link
Contributor

Ben-Ho commented Feb 23, 2018

strukturierung in den extra-ordner find ich gut, sonst vermischt sich symfony-sturktur-anforderung mit unseren. Aber die benennung im bundle passt noch nicht mit den best-practices von symfony zusammen.
siehe: https://symfony.com/doc/2.8/bundles/best_practices.html

sollte aber keine direkte auswirkung auf diesen merge-request haben.

@Ben-Ho

Ben-Ho approved these changes Feb 23, 2018

@kaufmo kaufmo merged commit ec8074c into 1.0 Feb 26, 2018

@kaufmo kaufmo deleted the psr4-subfolder branch Feb 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment