chore: sync alignment script with cuioss-parent-pom - #222
Conversation
Picks up cuioss/cuioss-parent-pom#1401: the check now also asserts that io.quarkus:quarkus-core resolves to the declared version.quarkus. version.quarkus drives quarkus-maven-plugin (a build extension) while the Quarkus artifacts come from whichever BOM is imported. Those are separate inputs and can drift apart, and the smallrye comparison stays silent when they do - both sides may still agree on smallrye. That drift is how the original outage began: cui-reference-documentation ran plugin 3.38.0 against BOM-supplied 3.39.0. Verified against this repo: ALIGNED, exit 0. These copies have to be kept in sync by hand - there is no shared parent to inherit the script from, which is the same reason version.quarkus is pinned locally here in the first place. Co-Authored-By: Claude <noreply@anthropic.com>
OliverWolffGIP
added
the
skip-bot-review
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Sorry @OliverWolffGIP, you've used your own review budget of 250,000 diff characters for the last 7 days.
You can request another review in 3 days and 10 hours by commenting @sourcery-ai review. Upgrade to get a review now.
|
Important Review skippedAuto reviews are limited based on label configuration. 🚫 Excluded labels (none allowed) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: cuioss/coderabbit/.coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
2 participants
Übernimmt cuioss/cuioss-parent-pom#1401.
Was sich ändert
Der Check prüft jetzt zusätzlich, dass
io.quarkus:quarkus-coreauf das deklarierteversion.quarkusauflöst.version.quarkussteuertquarkus-maven-plugin(eine Build-Extension), während die Quarkus-Artefakte aus der importierten BOM kommen. Das sind zwei getrennte Eingaben, die auseinanderlaufen können — und die smallrye-Prüfung bleibt dabei stumm, weil sich beide Seiten über smallrye durchaus einig sein können.Genau dieser Drift war der Auslöser des ursprünglichen Ausfalls:
cui-reference-documentationlief mit Plugin 3.38.0 gegen BOM-geliefertes 3.39.0.Verifikation
Gegen dieses Repo ausgeführt: ALIGNED, exit 0.
Das Script wurde in cuioss/cuioss-parent-pom#1401 gegen den echten Regressionsfall geprüft (erzwungener Plugin/BOM-Drift → MISALIGNED, exit 1).
Wartung
Diese Kopien müssen von Hand synchron gehalten werden — es gibt keinen gemeinsamen Parent, von dem das Script geerbt werden könnte. Das ist derselbe Grund, aus dem
version.quarkushier lokal gepinnt ist.