ICU-23091 Add Java source formatting configs and automated checks

Open echeran opened this issue 6 months ago • 8 comments

Add automated source formatting so that we don't have to worry about it again. In the future, the diffs that are visible in PRs will no longer have noise (extra diff lines) due to formatting unrelated to the code issues that are the focus of the PR.

Checklist

[X] Required: Issue filed: ICU-23091
[X] Required: The PR title must be prefixed with a JIRA Issue number. Example: "ICU-1234 Fix xyz"
[X] Required: Each commit message must be prefixed with a JIRA Issue number. Example: "ICU-1234 Fix xyz"
[X] Issue accepted (done by Technical Committee after discussion)
[ ] Tests included, if applicable
[X] API docs and/or User Guide docs changed or added, if applicable

ALLOW_MANY_COMMITS=true

Jun 05 '25 01:06 echeran

Notice: the branch changed across the force-push!

icu4j/demos/src/main/java/com/ibm/icu/dev/demo/calendar/CalendarApp.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/calendar/CalendarCalc.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/calendar/CalendarFrame.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/calendar/CalendarPanel.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/charsetdet/DetectingViewer.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/holiday/HolidayBorderPanel.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/holiday/HolidayCalendarDemo.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/impl/AppletFrame.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/impl/DemoApplet.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/impl/DemoTextBox.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/impl/DemoUtility.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/impl/DumbTextComponent.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/impl/Selection.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/Launcher.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/number/CurrencyDemo.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/rbnf/RbnfDemo.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/rbnf/RbnfSampleRuleSets.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/translit/AnyTransliterator.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/translit/CaseIterator.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/translit/Demo.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/translit/DemoApplet.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/translit/InfoDialog.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/translit/IntDiffer.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/translit/TransliteratingTextComponent.java is now changed in the branch
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/translit/TransliterationChart.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/Charset88591.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetASCII.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetBOCU1.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetCallback.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetCESU8.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetCompoundText.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetDecoderICU.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetEncoderICU.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetHZ.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetICU.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetISCII.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetISO2022.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetLMBCS.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetMBCS.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetProviderICU.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetSCSU.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetSelector.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetUTF16.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetUTF16BE.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetUTF16LE.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetUTF32.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetUTF32BE.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetUTF32LE.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetUTF7.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetUTF8.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/UConverterAlias.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/UConverterAliasDataReader.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/UConverterConstants.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/UConverterDataReader.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/UConverterSharedData.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/UConverterStaticData.java is now changed in the branch
icu4j/main/charset/src/main/java/com/ibm/icu/charset/UTF8.java is now changed in the branch
icu4j/main/charset/src/test/java/com/ibm/icu/dev/test/charset/TestCharset.java is now changed in the branch
icu4j/main/charset/src/test/java/com/ibm/icu/dev/test/charset/TestConversion.java is now changed in the branch
icu4j/main/charset/src/test/java/com/ibm/icu/dev/test/charset/TestSelection.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/BOCSU.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/Collation.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationBuilder.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationCompare.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationData.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationDataBuilder.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationDataReader.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationFastLatin.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationFastLatinBuilder.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationFCD.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationIterator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationKeys.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationLoader.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationRoot.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationRootElements.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationRuleParser.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationSettings.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationTailoring.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationWeights.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/ContractionsAndExpansions.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/FCDIterCollationIterator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/FCDUTF16CollationIterator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/IterCollationIterator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/SharedObject.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/TailoredSet.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/UTF16CollationIterator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/UVector32.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/UVector64.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/impl/text/RbnfScannerProviderImpl.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/AlphabeticIndex.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/CollationElementIterator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/CollationKey.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/Collator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/CollatorServiceShim.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/RawCollationKey.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/RuleBasedCollator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/SearchIterator.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/text/StringSearch.java is now changed in the branch
icu4j/main/collate/src/main/java/com/ibm/icu/util/GlobalizationPreferences.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/AlphabeticIndexTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationAPITest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationChineseTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationCreationMethodTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationCurrencyTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationDummyTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationEnglishTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationFinnishTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationFrenchTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationFrozenMonkeyTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationGermanTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationIteratorTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationKanaTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationMiscTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationMonkeyTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationRegressionTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationServiceTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationSpanishTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationThaiTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationThreadTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationTurkishTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/Counter.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/G7CollationTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/LotusCollationKoreanTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/UCAConformanceTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/format/GlobalizationPreferencesTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/format/RbnfLenientScannerTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/search/SearchTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/util/ICUResourceBundleCollationTest.java is now changed in the branch
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/util/ULocaleCollationTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/calendar/DataDrivenCalendarTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/CompactDecimalFormatTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/DataDrivenFormatTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/DateFormatTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/IntlTestDecimalFormatAPIC.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/IntlTestDecimalFormatSymbolsC.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/MeasureUnitCompatibilityTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/MeasureUnitGeneratorTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/MeasureUnitTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/NumberFormatDataDrivenTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/NumberFormatRegressionTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/NumberFormatSpecificationTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/NumberFormatTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/NumberRegressionTests.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/PluralRangesTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/PluralRulesTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/RbnfTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/TestMessageFormat.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/TimeZoneFormatTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/message2/Mf2FeaturesTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/normalizer/BasicTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/ModifierTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/NumberFormatterApiTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/NumberParserTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/NumberPermutationTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/NumberRangeFormatterTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/PatternStringTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/PropertiesTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/rbbi/LSTMBreakEngineTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/CalendarHandler.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/CompatibilityTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/CoverageTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/ExceptionHandler.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/FormatHandler.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/SerializableChecker.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/SerializableTestUtility.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/SerializableWriter.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/stringprep/TestIDNARef.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/stringprep/TestStringPrep.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/TestLocaleNamePackaging.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/util/CurrencyTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/util/ICUResourceBundleTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/util/ICUServiceTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/util/LocaleDataTest.java is now changed in the branch
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/util/ULocaleTest.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Assert.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/BMPSet.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/BurmeseBreakEngine.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/BytesDictionaryMatcher.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/CharsDictionaryMatcher.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/CjkBreakEngine.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/DictionaryBreakEngine.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/DictionaryData.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/DictionaryMatcher.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/KhmerBreakEngine.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/LSTMBreakEngine.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CacheBase.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CacheValue.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CalendarAstronomer.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CalendarCache.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CalendarUtil.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CaseMapImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CharacterIteration.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CharacterIteratorWrapper.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CharacterPropertiesImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CharTrie.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ClassLoaderUtil.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CollectionSet.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CSCharacterIterator.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/CurrencyData.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/DateNumberFormat.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/DayPeriodRules.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/DontCareFieldPosition.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/EmojiProps.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/EraRules.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/FormattedStringBuilder.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/FormattedValueFieldPositionIteratorImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/FormattedValueStringBuilderImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Grego.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUBinary.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUCache.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUConfig.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUData.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUDataVersion.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUDebug.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICULocaleService.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUNotifier.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUResourceBundle.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUResourceBundleImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUResourceBundleReader.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUResourceTableAccess.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICURWLock.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUService.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/IDNA2003.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/IllegalIcuArgumentException.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ImmutableEntry.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/IntTrie.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/IntTrieBuilder.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/InvalidFormatException.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/IterableComparator.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/JavaTimeConverters.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/JavaTimeZone.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/LocaleDisplayNamesImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/LocaleFallbackData.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/LocaleIDParser.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/LocaleIDs.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/LocaleUtility.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Norm2AllModes.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Normalizer2Impl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/OlsonTimeZone.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Pair.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/PatternProps.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/PatternTokenizer.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/PluralRulesLoader.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/PropsVectors.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Punycode.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/PVecToTrieCompactHandler.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/RBBIDataWrapper.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Relation.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/RelativeDateFormat.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ReplaceableUCharacterIterator.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ResourceBundleWrapper.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Row.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/RuleCharacterIterator.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/SimpleCache.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/SimpleFilteredSentenceBreakIterator.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/SimpleFormatterImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/SoftCache.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/SortedSetRelation.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/StandardPlural.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/StaticUnicodeSets.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/StringPrepDataReader.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/StringRange.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/StringSegment.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/TextTrieMap.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/TimeZoneAdapter.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/TimeZoneGenericNames.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/TimeZoneNamesFactoryImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/TimeZoneNamesImpl.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Trie.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Trie2_16.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Trie2_32.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Trie2.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Trie2Writable.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/TrieBuilder.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/TrieIterator.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/TZDBTimeZoneNames.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UBiDiProps.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCaseProps.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharacterIteratorWrapper.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharacterName.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharacterNameChoice.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharacterNameReader.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharacterProperty.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharacterUtility.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharArrayIterator.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UnicodeMap.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UnicodeMapIterator.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UnicodeRegex.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UnicodeSetStringSpan.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UPropertyAliases.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UResource.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/URLHandler.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/USerializedSet.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/Utility.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/UTS46.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ValidIdentifiers.java is now changed in the branch
icu4j/main/core/src/main/java/com/ibm/icu/impl/ZoneMeta.java is now changed in the branch
icu4j/pom.xml is no longer changed in the branch

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

Jun 05 '25 17:06 jira-pull-request-webhook[bot]

Notice: the branch changed across the force-push!

.github/workflows/icu4j.yml is different

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

Jun 05 '25 17:06 jira-pull-request-webhook[bot]

For review purposes and consideration:

There are 2 commits in the PR: one has the key config changes and doc updates, and the other has the changes to the remaining codebase Java source files
Let's decouple the work of reviewing/approving from the action of actually merging the PR, and not assume that we do these things at the same time for this PR
When this gets merged, then any concurrent open PRs that touch Java files will need to update themselves with the latest on main to pick up the new source formatting. That will cause churn. We can choose the right time to merge. Of course, likewise, this PR will need to pick up the latest from main for any Java changes between now and when we actually merge

Jun 05 '25 17:06 echeran

@yumaoka @richgillam FYI

Jun 05 '25 19:06 markusicu

thanks!
what about the 11 failing ("cancelled") checks?
config looks plausible
code reformatting changes... I know not to quibble with the formatter...
we will need to talk about when to merge this
we might want to push everyone (mea culpa etc.) to move on pending pull requests and reviews, so that there are fewer pending when this goes in

Jun 05 '25 20:06 markusicu

what about the 11 failing ("cancelled") checks?

I retried them and all but one succeeded. The ones that passed after retrying were all from the ICU4C CI workflow, and given that this PR here is only about source code formatting for Java, it's safe to conclude that those ICU4C check failures during the first attempt are unrelated. The other check that failed is the "meta check" job called "Wait for Required Checks", and it fails because the single-commit check fails. For now, I am leaving the two commits in the PR to remain separate in order to make it easier to understand when reviewing.

Jun 05 '25 21:06 echeran

The other check that failed is the "meta check" job called "Wait for Required Checks", and it fails because the single-commit check fails. For now, I am leaving the two commits in the PR to remain separate in order to make it easier to understand when reviewing.

We should add the magic incantation that allows us to rebase+merge the two commits.

Jun 05 '25 22:06 markusicu

We should add the magic incantation that allows us to rebase+merge the two commits.

Done.

Jun 05 '25 23:06 echeran

Notice: the branch changed across the force-push!

.github/workflows/icu4j.yml is different
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/calendar/CalendarFrame.java is different
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/calendar/CalendarPanel.java is different
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/charsetdet/DetectingViewer.java is different
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/holiday/HolidayBorderPanel.java is different
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/holiday/HolidayCalendarDemo.java is different
icu4j/demos/src/main/java/com/ibm/icu/dev/demo/Launcher.java is different
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetISO2022.java is different
icu4j/main/charset/src/main/java/com/ibm/icu/charset/CharsetMBCS.java is different
icu4j/main/charset/src/test/java/com/ibm/icu/dev/test/charset/TestCharset.java is different
icu4j/main/charset/src/test/java/com/ibm/icu/dev/test/charset/TestConversion.java is different
icu4j/main/charset/src/test/java/com/ibm/icu/dev/test/charset/TestSelection.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationBuilder.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/impl/coll/CollationFCD.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/text/AlphabeticIndex.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/text/CollationElementIterator.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/text/Collator.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/text/CollatorServiceShim.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/text/RuleBasedCollator.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/text/StringSearch.java is different
icu4j/main/collate/src/main/java/com/ibm/icu/util/GlobalizationPreferences.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationAPITest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationIteratorTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationMiscTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationRegressionTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationThaiTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/CollationThreadTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/Counter.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/collator/UCAConformanceTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/format/GlobalizationPreferencesTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/format/RbnfLenientScannerTest.java is different
icu4j/main/collate/src/test/java/com/ibm/icu/dev/test/search/SearchTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/calendar/DataDrivenCalendarTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/CompactDecimalFormatTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/DataDrivenFormatTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/DateFormatTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/IntlTestDecimalFormatAPIC.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/IntlTestDecimalFormatSymbolsC.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/MeasureUnitGeneratorTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/MeasureUnitTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/NumberFormatTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/RbnfTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/TestMessageFormat.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/format/TimeZoneFormatTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/message2/Mf2FeaturesTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/normalizer/BasicTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/NumberFormatterApiTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/NumberParserTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/NumberPermutationTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/number/NumberRangeFormatterTest.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/ExceptionHandler.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/serializable/FormatHandler.java is different
icu4j/main/common_tests/src/test/java/com/ibm/icu/dev/test/util/ULocaleTest.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/DictionaryBreakEngine.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/breakiter/LSTMBreakEngine.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/CharacterIteratorWrapper.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/CSCharacterIterator.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/DateNumberFormat.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/EraRules.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/FormattedValueStringBuilderImpl.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/ICUDebug.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/ImmutableEntry.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/JavaTimeConverters.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/JavaTimeZone.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/LocaleFallbackData.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/OlsonTimeZone.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/PatternTokenizer.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/PluralRulesLoader.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/RBBIDataWrapper.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/RelativeDateFormat.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/ReplaceableUCharacterIterator.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/Row.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/SimpleFilteredSentenceBreakIterator.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/StaticUnicodeSets.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/TimeZoneAdapter.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/Trie2.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCaseProps.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharacterIteratorWrapper.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharacterProperty.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/UCharArrayIterator.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/UnicodeMap.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/UnicodeSetStringSpan.java is different
icu4j/main/core/src/main/java/com/ibm/icu/impl/Utility.java is different

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

Oct 31 '25 20:10 jira-pull-request-webhook[bot]

@markusicu Ready again when you are. I don't see any PRs that simultaneously: 1) touch ICU4J 2) are ready to merge 3) are recent enough to consider. The only possible exception is maybe #3468, but that's very small, enough so to not hold up for.

Oct 31 '25 21:10 echeran

@markusicu Ready again when you are.

Ok... :-)

I reminded myself that the TC is on board with this now:

2025-08-21: When do we start auto-formatting the Java source code? Mihai proposes that we do this right after the release (or else ASAP) / after merging maint-78 to main [...] Agreed
2025-07-17: Consensus: Let’s do this before 78 release front loading – soon after aug07
2025-06-12: Let’s decide when a good time to merge is. Preferably a time when is least disruptive to us & PR contributors

I don't see any PRs that simultaneously: 1) touch ICU4J 2) are ready to merge 3) are recent enough to consider.

The only possible exception is maybe #3468, but that's very small, enough so to not hold up for.

And it already has a merge conflict.

@richgillam @mihnita @yumaoka FYI

Nov 04 '25 23:11 markusicu

@echeran if you reversed the commits, it would be easier to keep the GitHub UI alive to get to the approve button...

Nov 04 '25 23:11 markusicu