Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Make some classes final#50660

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
lyrixx wants to merge1 commit intosymfony:7.0fromlyrixx:final
Closed

Make some classes final#50660

lyrixx wants to merge1 commit intosymfony:7.0fromlyrixx:final

Conversation

@lyrixx
Copy link
Member

QA
Branch?7.0
Bug fix?no
New feature?no
Deprecations?no
Tickets
LicenseMIT
Doc PR

dunglas reacted with thumbs up emoji
derrabus added a commit that referenced this pull requestJun 22, 2023
…zer (lyrixx)This PR was merged into the 6.3 branch.Discussion----------[Serializer] Refactor tests to not extends ObjectNormalizer| Q             | A| ------------- | ---| Branch?       | 6.3| Bug fix?      | no| New feature?  | no| Deprecations? | no| Tickets       || License       | MIT| Doc PR        |see#50660 (comment)Commits-------2269c3f [Serializer] Refactor tests to not extends ObjectNormalizer
@nicolas-grekas
Copy link
Member

(rebase needed)

@lyrixx
Copy link
MemberAuthor

I need to finish#50736 first

@nicolas-grekas
Copy link
Member

Thank you@lyrixx.

nicolas-grekas added a commit that referenced this pull requestJun 30, 2023
This PR was merged into the 7.0 branch.Discussion----------Make some classes final| Q             | A| ------------- | ---| Branch?       | 7.0| Bug fix?      | no| New feature?  | no| Deprecations? | no| Tickets       || License       | MIT| Doc PR        |Commits-------f39ede4 Make some classes final
@lyrixx
Copy link
MemberAuthor

This is weird, the PR has not been closed?!

@nicolas-grekas
Copy link
Member

🤷

sstok reacted with laugh emoji

@lyrixxlyrixx deleted the final branchJune 30, 2023 22:30
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nicolas-grekasnicolas-grekasnicolas-grekas left review comments

@stofstofstof left review comments

@derrabusderrabusderrabus left review comments

@dunglasdunglasAwaiting requested review from dunglasdunglas is a code owner

@ycerutoycerutoAwaiting requested review from yceruto

Assignees

No one assigned

Projects

None yet

Milestone

7.0

Development

Successfully merging this pull request may close these issues.

5 participants

@lyrixx@nicolas-grekas@stof@derrabus@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp