search

frankframework / frank-doc

Frank!Doc
https://frankdoc.frankframework.org
Apache License 2.0
4 stars 5 forks source link

Improve @FrankDocGroup annotation #175

Closed mhdirkse closed 4 months ago

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.09%. Comparing base (19f1186) to head (2055077). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #175 +/- ## ============================================ - Coverage 84.40% 84.09% -0.31% + Complexity 1392 1388 -4 ============================================ Files 79 79 Lines 4457 4446 -11 Branches 523 526 +3 ============================================ - Hits 3762 3739 -23 - Misses 563 571 +8 - Partials 132 136 +4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mhdirkse commented 4 months ago

Wacht nog even met mergen. Ik moet de enum constanten ook bijwerken in mijn PR op het F!F.

mhdirkse commented 4 months ago

Bedankt @jkosternl. De build kan niet slagen omdat hij probeert of de F!F sources verwerkt kunnen worden door de te pushen versie van de Frank!Doc. Deze F!F sources hebben nog de oude-stijl @FrankDocGroup annotaties. Ik check wel of de build niet om een andere reden faalt.

nielsm5 commented 4 months ago

@mhdirkse ik neem aan dat wij deze gewoon kunnen mergen? Het is dan wel handig om het versienummertje van 3.0-snapshot op te hogen naar 3.1-snapshot, dit is fout gegaan tijdens de vorige release.

mhdirkse commented 4 months ago

Ja, klopt. Als je hem released met een nieuw versienummer, dan gaat de build van het Frank!Framework niet stuk omdat het F!F dan nog naar de oude wijst. Na deze stap moet je F!F PR https://github.com/frankframework/frankframework/pull/6389 uitbreiden zodat hij naar de nieuwe Frank!Doc versie wijst. Als je die PR met deze wijziging merged dan zou alles goed moeten verlopen, met één kanttekening. De build van de Frank!Doc zal falen op GitHub zolang het F!F nog niet de nieuwe @FrankDocGroup annotatie gebruikt. Je moet deze PR dus mergen ondanks een falende build.

nielsm5 commented 4 months ago

Done, @mhdirkse update jij de versie in de FF pom?