8000 Introduce 3.1 and 4.0 upgrade files by xabbuh · Pull Request #17583 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Introduce 3.1 and 4.0 upgrade files #17583

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
Jan 29, 2016
Merged

Conversation

xabbuh
Copy link
Member
@xabbuh xabbuh commented Jan 27, 2016
Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

Form
----

* The `choices_as_values` option of the `ChoiceType` has been deprecated and will be removed in Symfony 4.0.
Copy link
Member

Choose a reason for hiding this comment

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

In this line I miss the "... use XXX instead" Or is it a removed feature without alternative?

Copy link
Member Author

Choose a reason for hiding this comment

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

There is no alternative. The option only still exists as a remainer of the choices transformation done in Symfony 2.7.

Copy link
Member

Choose a reason for hiding this comment

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

This is a no-op option in 3.x

@javiereguiluz javiereguiluz changed the title introduce 3.1 and 4.0 upgrade files Introduce 3.1 and 4.0 upgrade files Jan 28, 2016
@fabpot
Copy link
Member
fabpot commented Jan 28, 2016

Can you add line breaks like done in the other UPGRADE files?

@xabbuh
Copy link
Member Author
xabbuh commented Jan 28, 2016

@fabpot Sorry, not sure I understand what you mean.

@fabpot
Copy link
Member
fabpot commented Jan 28, 2016

I mean wrapping at around 80 chars.

@xabbuh
Copy link
Member Author
xabbuh commented Jan 28, 2016

Oh sure, it seems I wrapped around 100 characters instead.

@javiereguiluz
Copy link
Member

👍
Status: Reviewed

@fabpot
Copy link
Member
fabpot commented Jan 29, 2016

Thank you @xabbuh.

@fabpot fabpot merged commit f68e94a into symfony:master Jan 29, 2016
fabpot added a commit that referenced this pull request Jan 29, 2016
This PR was merged into the 3.1-dev branch.

Discussion
----------

Introduce 3.1 and 4.0 upgrade files

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

Commits
-------

f68e94a introduce 3.1 and 4.0 upgrade files
@xabbuh xabbuh deleted the upgrade-file branch January 29, 2016 08:22
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.

5 participants
0