Skip to content

fix(tooling): use LinkedHashSet for deterministic option ordering (backport to 4.4.x)#2337

Merged
jbonofre merged 1 commit into
karaf-4.4.xfrom
cherry-pick-2306-karaf-4.4.x
Mar 14, 2026
Merged

fix(tooling): use LinkedHashSet for deterministic option ordering (backport to 4.4.x)#2337
jbonofre merged 1 commit into
karaf-4.4.xfrom
cherry-pick-2306-karaf-4.4.x

Conversation

@jbonofre
Copy link
Copy Markdown
Member

Summary

…help generation (#2306)

* Revert "Use single file generation to avoid verify race condition (#2285)"

This reverts commit 3b067c9.

* fix(tooling): use LinkedHashSet for deterministic option ordering in help generation

Replace HashSet with LinkedHashSet in all CommandHelpPrinter
implementations (Markdown, AsciiDoctor, DocBook, UserConf) to
preserve insertion order of options.

HashSet provides no ordering guarantee, causing non-deterministic
output that breaks the test-commands-generate-help invoker test
when dependencies change (e.g. jline upgrade from 3.30.6 to 3.30.8).
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 14, 2026

Test Results

242 files  ±0  242 suites  ±0   20m 36s ⏱️ -6s
993 tests ±0  950 ✅ ±0  43 💤 ±0  0 ❌ ±0 
997 runs  ±0  954 ✅ ±0  43 💤 ±0  0 ❌ ±0 

Results for commit 81b3b48. ± Comparison against base commit 15822d1.

♻️ This comment has been updated with latest results.

@jbonofre jbonofre merged commit ebd224b into karaf-4.4.x Mar 14, 2026
4 of 5 checks passed
@jbonofre jbonofre deleted the cherry-pick-2306-karaf-4.4.x branch March 14, 2026 18:09
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.

1 participant