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

[V3] Fix messages for duplicate languages #301

Merged

Conversation

mristin
Copy link
Contributor

@mristin mristin commented Jan 26, 2024

We fix the confusing messages of the constraints related to language strings. Namely, "Xxx specifies no duplicate languages" can be misunderstood such that Xxx should actually specify duplicate languages, but does not.

Obviously, we want to specify unique languages in such properties.

Fixes #299.

We fix the confusing messages of the constraints related to language
strings. Namely, "Xxx specifies no duplicate languages" can be
misunderstood such that `Xxx` should actually specify duplicate
languages, but does not.

Obviously, we want to specify unique languages in such properties.

Fixes #299.
@mristin mristin force-pushed the mristin/Fix-constraint-message-for-duplicate-languages-in-V3 branch from 6f9e58e to 86da181 Compare January 26, 2024 11:31
@mristin mristin changed the title [V3] Fix message for duplicate languages [V3] Fix messages for duplicate languages Jan 26, 2024
@mristin mristin merged commit c9692bc into main Jan 26, 2024
2 checks passed
@mristin mristin deleted the mristin/Fix-constraint-message-for-duplicate-languages-in-V3 branch January 26, 2024 11:34
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.

Misleading error message on validation of MultiLanguageProperty
1 participant