Improve @FrankDocGroup annotation

[mhdirkse]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

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

Additional details and impacted files

@@             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     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nielsm5]

Fijn dat je ook de readme aan hebt gepast!

[jkosternl]

Looking good Martijn! Good job.
I've put some minor remarks mostly, if you can address those, I'd appreciate it.

[mhdirkse]

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

[jkosternl]

Thanks Martijn. Good improvements. I already approve it, assuming you're going to check the failing build 😉

[mhdirkse]

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]

@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]

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 frankframework/frankframework#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]

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