Skip to content
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

Should FOSUserBundle3 follow current Symfony best practices #2

Open
ahundiak opened this issue Feb 26, 2016 · 2 comments
Open

Should FOSUserBundle3 follow current Symfony best practices #2

ahundiak opened this issue Feb 26, 2016 · 2 comments

Comments

@ahundiak
Copy link
Member

When Symfony 2 was first released there was a set of informal guidelines for developing bundles which the FOSUserBundle followed fairly closely.

We now have an official set of best practices: http://symfony.com/doc/current/best_practices/index.html

Quite a few changes.

So do we go ahead and refactor the code to follow the new guidelines as much as practical while maintaining backwards compatibility? Or pretty much just leave it as is for now?

@alexislefebvre
Copy link

While using guidelines is a good thing, may it prevent merges to upstream because it would change a lot of code?

@ahundiak
Copy link
Member Author

The master branch is already following some of the newer practices so I don't really anticipate a big problem with merging if that is the final direction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants