Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fjerne/erstatte avhengighet til javax.validation #58

Closed
arakvaag opened this issue Jan 2, 2023 · 5 comments
Closed

Fjerne/erstatte avhengighet til javax.validation #58

arakvaag opened this issue Jan 2, 2023 · 5 comments

Comments

@arakvaag
Copy link

arakvaag commented Jan 2, 2023

NoCommons har en avhengighet til javax.validation - validation-api. Siste versjon av dette biblioteket er fra desember 2017, og er del av Java EE (javax-namespacet). Nå har jakarta-namespacet tatt over, også for validation-api.

Vi synes det er uheldig at NoCommons drar inn en gammel Java EE-avhengighet inn i våre applikasjoner. Vi tenker at det kanskje er en god ide å bygge om slik at bruken av denne validation-api fjernes, selv om vi forstår at det er en større endring. Om det er ønskelig å beholde API-et er det vel da viktig å få oppgradert til jakarta-namespacet.

Kan dere si noe om deres vurdering av problemet, og om dere ser mulighet for at det blir gjort oppgradering av biblioteket som løser det?

@eivinhb
Copy link
Member

eivinhb commented Jan 9, 2023

Det vel egentlig ikke noe problem å oppgradere til hibernate 6 og bruke jakarta. Men siden det er breaking så må vi kanskje lage en ny versjon ala et Jetty har holdt på med en stund.
Tiden er vel inne for at vi gjør det her også. Jeg ser @janhoy har foreslått å gå over på java 11 #61 Kan hende vi kan lage en ny versjon på java 17 (?) og gjøre det totale spranget samtidig.

@eivinhb
Copy link
Member

eivinhb commented Jan 11, 2023

@arakvaag For dere, ville det være et problem å lage en ny release med target java 17? ref: diskusjonen her: #61

@eivinhb
Copy link
Member

eivinhb commented Jan 11, 2023

Når jeg tenker meg om så er det kanskje egentlig ikke noe problem å bytte ut javax.validation med jakarta.validation og fortsatt bruk av java 8. Det er jo en major breaking change. Sånn sett kunne vi kanskje bare releaset 1.0 med jakarta.
Egentlig hadde jeg håpet at 1.0-releasen gikk på å få java.time inn i stedet for å bruke Calender-apiet som biblioteket bruker i dag.

@eivinhb
Copy link
Member

eivinhb commented Jan 11, 2023

Laget denne PR-en:
#64

@eivinhb
Copy link
Member

eivinhb commented Jan 16, 2023

Releaset 0.14.0

@eivinhb eivinhb closed this as completed Jan 16, 2023
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

No branches or pull requests

2 participants