Skip to content

[PhpUnitBridge] Enforce @-silencing of deprecation notices according to new policy #15031

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 19, 2015

Conversation

nicolas-grekas
Copy link
Member

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR symfony/symfony-docs#5413

@stof
Copy link
Member

stof commented Jun 19, 2015

This may be an issue for deprecations triggered in other libraries than Symfony though, if they don't use the new silencing pattern

@nicolas-grekas
Copy link
Member Author

Yes and no: if the deprecations where in place before the bridge was released, then these libs already have a way to make their users handle them. If the deprecations where added by taking inspiration from Symfony, then this is a recent move and I'd advocate we need to push them one step forward.
It took us several months to create the current deprecation policy and I now really like it. This bridge should help spread the word :)

@stof
Copy link
Member

stof commented Jun 19, 2015

Well, pushing to the new way is indeed a good idea. 👍

You should also update http://symfony.com/doc/current/contributing/code/conventions.html#deprecations with the new convention btw

@nicolas-grekas
Copy link
Member Author

Doc PR is symfony/symfony-docs#5413

@fabpot
Copy link
Member

fabpot commented Jun 19, 2015

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 9eea9eb into symfony:2.7 Jun 19, 2015
fabpot added a commit that referenced this pull request Jun 19, 2015
… according to new policy (nicolas-grekas)

This PR was merged into the 2.7 branch.

Discussion
----------

[PhpUnitBridge] Enforce @-silencing of deprecation notices according to new policy

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

Commits
-------

9eea9eb [PhpUnitBridge] Enforce @-silencing of deprecation notices according to new policy
@nicolas-grekas nicolas-grekas deleted the deprec-strict branch July 1, 2015 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants