Skip to content

[FrameworkBundle] Cleanup #33635

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
Sep 19, 2019
Merged

[FrameworkBundle] Cleanup #33635

merged 1 commit into from
Sep 19, 2019

Conversation

yceruto
Copy link
Member

@yceruto yceruto commented Sep 18, 2019

Q A
Branch? master
Bug fix? no
New feature? no
Deprecations? no
Tickets -
License MIT
Doc PR -

These PHP templates are no longer used after removing the PHP Templating layer.

Copy link
Member

@fabpot fabpot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one might be controversial. Even if these templates are not used anymore, they might be useful for people not using the full-stack framework. I suppose these can also be moved to a new PHP template bundle if some want to still use PHP as a templating language.

@fabpot
Copy link
Member

fabpot commented Sep 19, 2019

Thank you @yceruto.

fabpot added a commit that referenced this pull request Sep 19, 2019
This PR was merged into the 5.0-dev branch.

Discussion
----------

[FrameworkBundle] Cleanup

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

These PHP templates are no longer used after removing the PHP Templating layer.

Commits
-------

f1ebb04 Cleanup
@fabpot fabpot merged commit f1ebb04 into symfony:master Sep 19, 2019
@yceruto yceruto deleted the cleanup branch September 19, 2019 11:08
@fabpot fabpot mentioned this pull request Nov 12, 2019
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.

4 participants