-
Notifications
You must be signed in to change notification settings - Fork 0
Reduce dependency footprint of spring data kotlin helper #51
Conversation
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.
Version 31-head-5930aec-123543-SNAPSHOT
was generated from commit 5930aec8d1065062a5c2379803c5684de2fe8f8b
5930aec
to
8e7888b
Compare
@@ -36,6 +40,7 @@ | |||
try { | |||
return Optional.of(wrappedParser.apply(str)); | |||
} catch (NumberFormatException e) { | |||
LOGGER.error("Bad config property value [{}] was supposed to be a number but wasn't parseable", str, e); |
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.
This was changed to keep the static analysis happy.
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.
Version 31-head-8e7888b-123753-SNAPSHOT
was generated from commit 8e7888b74d229e62e3b24ea3ef45f43bba9a7b0b
should remain draft until we've verified it works with downstream projects |
suppression.xml
Outdated
@@ -19,12 +19,12 @@ | |||
]]></notes> | |||
<vulnerabilityName regex="true">CVE-2019-14900|CVE-2020-25638</vulnerabilityName> | |||
</suppress> | |||
<suppress until="2021-03-01Z"> | |||
<suppress until="2021-04-01Z"> |
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.
Does this make sense?
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.
No. It doesn't. It might have done a month ago when I first opened this PR. Sigh.
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.
Version 31-head-d525366-134432-SNAPSHOT
was generated from commit d525366c0be41f77298128f6cd11cdb404309be2
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.
Version 31-head-1442509-134438-SNAPSHOT
was generated from commit 144250965b6217a362fadb56ff4a428110001c4e
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.
Version 31-3c80f61-134475
was generated from commit 3c80f61397ae71e05da50af253dddd01a3440d82
…l-id-refactor Reduce dependency footprint of spring data kotlin helper
Our kotlin version of Spring Data Persistable had an unnecessary dependency on a spring boot starter rather than just the Spring Data jar. It also could have had the dependency as provided to avoid dependency clashes.