Add means to check if version is a snapshot programatically. #749
Conversation
…edVersion Introduces a method, getDecoratedVersion() in the VersionConfig class, allowing you to easily retrieve the cached decorated version, instead of having to do versionProvider().get(), which looks less nice (imo) Also introduces a method, isSnapshot() to DecoratedVersion, which allows you to check if the decorated version is a snapshot. This is passed the value of VersionContact.isSnapshot(). This is convenient for checking if a version is a snapshot or not when building the project.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #749 +/- ##
============================================
+ Coverage 61.00% 62.74% +1.74%
- Complexity 398 404 +6
============================================
Files 81 81
Lines 1554 1530 -24
Branches 147 148 +1
============================================
+ Hits 948 960 +12
+ Misses 535 504 -31
+ Partials 71 66 -5 ☔ View full report in Codecov by Sentry. |
|
Do we need 100% code coverage, or no? |
|
@bgalek (you seem to be the main person merging prs) |
|
hi! coverage is not a target - but having a tests is ;) Please add at least one test of this feature to document the behaviour |
|
it's mainly just exposing a couple of getters tbh, not anything very interesting, so idk how I'd go about testing that without said tests being effectively useless lol also, could you take a peek at my other pr? because, I'm looking to get some feedback/opinions on it |
|
@bgalek yo, unsure if you saw this or not bc it was a weekend, so I'm gonna bump it lol |
Exposes cached
DecoratedVersion, and introduce isSnapshot() toDecoratedVersion.Fixes #678.
getDecoratedVersion()in theVersionConfigclass, allowing you to easily retrieve the cached decorated version, instead of having to doversionProvider().get(), which looks less nice (imo) (and is especially nice in groovy/kotlin code where you'll dodecoratedVersioninstead ofversionProvider().get())isSnapshot()toDecoratedVersion, which allows you to check if the decorated version is a snapshot. This is passed the value ofVersionContact.isSnapshot(). This is convenient for checking if a version is a snapshot or not when building the project.