Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update Maven plugins, dependencies and improve code #12

Merged
merged 2 commits into from
Feb 13, 2024

Conversation

driesva
Copy link
Contributor

@driesva driesva commented Feb 12, 2024

Omschrijving

Hoofdreden voor de wijziging is dat er voor enkele van de transitive dependencies (e.g. GeoTools) vulnerabilities zijn gerapporteerd.
In deze PR daarom version bumps en hier en daar wat code verbeteringen.

Type verandering

  • Minor change
    • GeoTools 29.x
    • maven plugins naar laatste versie
    • maven enforcer plugin introduceren
    • downgrade jakarta.xml.bind-api versie (moet nog degene zijn die javax package bevat)
  • Refactor
    • @Override toevoegen
    • Gebruik van pattern matching for instanceof & casts weg waar niet nodig
    • final indien mogelijk

Checklist:

(niet relevante zaken heb ik ook aangevinkt)

  • Ik heb de code in deze PR zelf nogmaals nagekeken
  • Ik heb mijn code beter achtergelaten dan dat ik het aantrof
  • De code is leesbaar en de moeilijke onderdelen zijn voorzien van commentaar
  • Ik heb de tests toegevoegd/uitgebreid indien nodig
  • Ik heb de tests gedraaid die de werking van mijn wijziging bewijst
  • De PDOK documentatie is bijgewerkt indien nodig.
  • Er zit geen gevoelig informatie in deze PR (wachtwoorden etc)

* e.g. GeoTools 29.x
* Maven Enforcer plugin to have the correct maven version
* Code quality improvements: Override annotation, final if possible, no casting if not required, ...
Copy link
Collaborator

@roelarents roelarents left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx 👍

@roelarents roelarents merged commit 6af4635 into PDOK:master Feb 13, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants