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
chore: Remove jodd dependency (DEV-3069) #3024
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3024 +/- ##
===========================================
+ Coverage 11.74% 88.51% +76.77%
===========================================
Files 246 258 +12
Lines 22907 22846 -61
===========================================
+ Hits 2690 20223 +17533
+ Misses 20217 2623 -17594 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thanks!
41ffc9a
to
0e45b5c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the time zone you have choosen the ZoneId.systemDefault()
.
This will actually depend on the servers settings. IMHO it would be better to replace this with a private val utc = ZoneId.of("UTC")
for both places.
I have tested this out locally and the tests still pass.
This would make the code independent of external settings.
webapi/src/main/scala/org/knora/webapi/messages/util/DateUtil.scala
Outdated
Show resolved
Hide resolved
webapi/src/main/scala/org/knora/webapi/messages/util/DateUtil.scala
Outdated
Show resolved
Hide resolved
Java's implementation uses an days starting at midnight, not at noon, thankfully.
0e45b5c
to
9028ef7
Compare
Java's implementation uses an days starting at midnight, not at noon, thankfully. JulianFields docs
The tests seem to be good and failed when appropriate, so I have confidence in this change.
Pull Request Checklist
Task Description/Number
Issue Number: DEV-
PR Type
Basic requirements for bug fixes and features
Does this PR introduce a breaking change?
Does this PR change client-test-data?