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

Allow initial System.Text.Json.JsonSerializerOptions to be a copy #3207

Conversation

CptWesley
Copy link
Contributor

Resolves #3206

@jeremydmiller
Copy link
Member

@CptWesley Got it, looks fine when the build goes green. This will be in 7.13. I don't have any set time for that offhand

@CptWesley
Copy link
Contributor Author

CptWesley commented May 15, 2024

@jeremydmiller The failing test seems flaky, it passes locally most of the time, but fails occasionally (failed for the first time after 33 iterations when using Run Until Failure). Not sure if that was a known issue.

@jeremydmiller
Copy link
Member

Known aggravation. Only fails on CI, and very rarely outside of that one specific combo in the CI matrix

@jeremydmiller jeremydmiller merged commit d1ef0fd into JasperFx:master May 16, 2024
11 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.

Allow initial System.Text.Json.JsonSerializerOptions to be a copy from existing options
2 participants