Skip to content

[FrameworkBundle] Deprecate useless --no-prefix option #23161

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
Jun 14, 2017

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented Jun 13, 2017

Q A
Branch? 3.4
Bug fix? no
New feature? no
BC breaks? no
Deprecations? yes
Tests pass? yes
Fixed tickets n/a
License MIT
Doc PR n/a

It was a workaround, not needed since #21228. Let's deprecate it and remove it in 4.0.

@fabpot
Copy link
Member

fabpot commented Jun 14, 2017

Thank you @chalasr.

@fabpot fabpot merged commit f7afa77 into symfony:3.4 Jun 14, 2017
fabpot added a commit that referenced this pull request Jun 14, 2017
… (chalasr)

This PR was merged into the 3.4 branch.

Discussion
----------

[FrameworkBundle] Deprecate useless --no-prefix option

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | yes
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

It was a workaround, not needed since #21228. Let's deprecate it and remove it in 4.0.

Commits
-------

f7afa77 [FrameworkBundle] Deprecate useless --no-prefix option
@chalasr chalasr deleted the transl-rm-opt branch June 14, 2017 07:39
This was referenced Oct 18, 2017
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