8000 Sync "not implementing the method" deprecations messages by fancyweb · Pull Request #32866 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Sync "not implementing the method" deprecations messages #32866

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
Aug 1, 2019

Conversation

fancyweb
Copy link
Contributor
@fancyweb fancyweb commented Aug 1, 2019
Q A
Branch? 4.3
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

Suggested in #32747 (comment)

Useful for consistency and for future reference for similar messages.

@fancyweb fancyweb force-pushed the sync-not-implementing-method branch from 3bf8156 to f6fae1c Compare August 1, 2019 12:51
@nicolas-grekas nicolas-grekas added this to the 4.3 milestone Aug 1, 2019
@nicolas-grekas
Copy link
Member

Thank you @fancyweb.

@nicolas-grekas nicolas-grekas merged commit f6fae1c into symfony:4.3 Aug 1, 2019
nicolas-grekas added a commit that referenced this pull request Aug 1, 2019
… (fancyweb)

This PR was merged into the 4.3 branch.

Discussion
----------

Sync "not implementing the method" deprecations messages

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

Suggested in #32747 (comment)

Useful for consistency and for future reference for similar messages.

Commits
-------

f6fae1c Sync "not implementing the method" deprecations messages
@fancyweb fancyweb deleted the sync-not-implementing-method branch August 1, 2019 13:04
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
0