Netflix / maestro

Maestro: Netflix’s Workflow Orchestrator
Apache License 2.0
3.3k stars 202 forks source link

Refactor: Update build.gradle and fix import order in SelJodaDateTimePropertyTest to pass Spotless check #57

Closed HankGuo93 closed 3 months ago

HankGuo93 commented 3 months ago

Pull Request type

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

chore: Update build.gradle

fix: Correct import order to pass Spotless check in SelJodaDateTimePropertyTest

HankGuo93 commented 3 months ago

If your application works correctly without -Duser.country=US, then specifying the country is not mandatory. However, if your application relies on specific locale settings (such as date and currency formats), it is better to explicitly specify the country.

To ensure consistent behavior, it is advisable to set both the language and the country.

jun-he commented 3 months ago

@HankGuo93 can you remove the change in test jvmArgs from this PR and I can merge the other change about the import? Thanks.

HankGuo93 commented 3 months ago

I've removed the changes to the test JVM arguments. The PR now only includes the import changes. Thanks!