adutra commented on code in PR #886:
URL: https://github.com/apache/polaris/pull/886#discussion_r1930723749
##########
quarkus/service/build.gradle.kts:
##########
@@ -165,6 +165,8 @@ tasks.named<Test>("intTest").configure {
// Same issue as above: allow a java security manager after Java 21
// (this setting is for the application under test, while the setting above
is for test code).
systemProperty("quarkus.test.arg-line", "-Djava.security.manager=allow")
+ // force the locale, just in case the user is having another default locale
+ systemProperty("quarkus.default-locale", "en_US")
Review Comment:
OTOH, don't we want to have i18n in Polaris? French customers may be happier
reading error messages in French. Setting the locale to `en_US` sounds very...
US centric :-) Can't we adapt the tests instead?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
