Skip to content

Schema/culture option#499

Merged
gregsdennis merged 5 commits into
masterfrom
schema/culture-option
Aug 4, 2023
Merged

Schema/culture option#499
gregsdennis merged 5 commits into
masterfrom
schema/culture-option

Conversation

@gregsdennis
Copy link
Copy Markdown
Collaborator

Resolves #493

@m-adamkiewicz, this should resolve you. I don't think there are any breaking changes as the significant stuff is internal. Basically it adds EvaluationOptions.Culture that you can set instead of ErrorMessages.Culture. It will fall back to the static setting when the option isn't set.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 4, 2023

Test Results

       12 files  ±0         12 suites  ±0   12s ⏱️ +2s
18 473 tests +1  17 180 ✔️ +1  1 293 💤 ±0  0 ±0 
18 830 runs  +1  17 489 ✔️ +1  1 341 💤 ±0  0 ±0 

Results for commit 3aecd48. ± Comparison against base commit 02dc3b8.

♻️ This comment has been updated with latest results.

@m-adamkiewicz
Copy link
Copy Markdown

Thanks @gregsdennis, I have reviewed this PR and that's exactly what we need.
Great job and looking forward to the release.

@gregsdennis gregsdennis merged commit 67b39f6 into master Aug 4, 2023
@gregsdennis gregsdennis deleted the schema/culture-option branch August 4, 2023 06:58
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.

Support for parallel evaluation with different cultures

2 participants