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

Fix CS for form templates locations #6773

Closed
wants to merge 1 commit into from
Closed

Conversation

petk
Copy link
Contributor

@petk petk commented Jul 28, 2016

According to the best practices template lowercased snake_case should be used for all template directory and file names. This patch fixes some inconsistencies left from the previous such updates.

@javiereguiluz
Copy link
Member

I tried to rebase this PR but I couldn't because of the massive internal changes introduced today by Symfony Docs. @peterkokot in #6784 I've made the changes that you propose here ... and I fixed other minor errors found while applying these changes. Thank you!

@petk
Copy link
Contributor Author

petk commented Jul 29, 2016

Sounds good to me. Yes, I've seen the documentation refactoring. Good job. Than I'll closing this in favor of your PR. Thanks.

@petk petk closed this Jul 29, 2016
weaverryan added a commit that referenced this pull request Aug 2, 2016
This PR was merged into the 2.7 branch.

Discussion
----------

Fix CS for form templates locations

This replaces #6773 because I tried to rebase it but I couldn't. All the credit for these fixes should go to @peterkokot, the original author of the other pull request.

Commits
-------

7b116e9 Fix CS for form templates locations
@petk petk deleted the patch-forms branch September 17, 2016 21:42
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.

3 participants