Skip to content

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