Skip to content
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

Date deserialization format type(s) #952

Merged
merged 7 commits into from
Nov 1, 2024

Conversation

iKsSs
Copy link

@iKsSs iKsSs commented Oct 18, 2024

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

@iKsSs
Copy link
Author

iKsSs commented Oct 18, 2024

The new version v3.31 of JMS Serializer will be released on Monday - 21st Oct 2024

@scyzoryck
Copy link
Collaborator

@iKsSs Could you take a look at the failing test, please?

iksss-invia and others added 2 commits October 29, 2024 15:41
@iKsSs iKsSs force-pushed the date-deserialization branch from ebdf95e to 1b573e5 Compare October 29, 2024 14:41
@iKsSs
Copy link
Author

iKsSs commented Oct 29, 2024

@iKsSs Could you take a look at the failing test, please?

ofc, should be fixed..

@iKsSs
Copy link
Author

iKsSs commented Oct 30, 2024

OK, I fixed the missing 'use' in the test from the previous commit(s)

@iKsSs
Copy link
Author

iKsSs commented Oct 30, 2024

Every failed test in PHP7.4 is connected to serialization :(
@scyzoryck will you look at it?

@scyzoryck
Copy link
Collaborator

I will take a look - I think I know where the problem is :)

Add  doctrine/annotations into dev dependencies.
Use new Download coverage class.
@scyzoryck
Copy link
Collaborator

Thanks @iKsSs for contribution!

@scyzoryck scyzoryck merged commit 2eefc11 into schmittjoh:master Nov 1, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants