Proposal - add a timestamp to metadata of "uncommitted" versions

[s4nk]

Use case:
During development cycle I keep experimenting, re-building and re-running my service. I would like the created artifacts (Docker images in my case) to have unique versions, but without commiting every change.

Proposed solution:
Add an ISO 8601 timestamp to version metadata when repository is dirty in

reckon/reckon-core/src/main/java/org/ajoberstar/reckon/core/Reckoner.java

Line 122 in 0f980de

.orElse("uncommitted");

, e.g.:
0.1.0-beta.0.1+e06c68a863eb6ceaf889ee5802f478c10c1464bb.uncommitted.20180615T084953Z

It's fairly simple to implement, I can open a PR if you think it makes sense.

[s4nk]

Alternatively the "uncommitted" suffix could become a timestamp instead.

[ajoberstar]

Seems worth considering. #86 is going to change the build metadata a little anyway, could change this at the same time.

Part of the deal with that one was to keep the metadata short though. I could see a couple options for this:

1. 0.1.0-beta.0.1+e06c68a863eb6ceaf889ee5802f478c10c1464bb.uncommitted.20180615T084953Z
2. 0.1.0-beta.0.1+e06c68a863eb6ceaf889ee5802f478c10c1464bb.20180615T084953Z
3. 0.1.0-beta.0.1+uncommitted.20180615T084953Z
4. 0.1.0-beta.0.1+20180615T084953Z
5. 0.1.0-beta.0.1+uncommitted.1529052593 (use epoch seconds instead of ISO for brevity)

I like dropping the commit hash, since you're in an uncommitted state anyway.

However, this would also have to consider how Gradle sorts versions, which might mean we have to keep the hash.

[s4nk]

That's great, thanks!

…

On Wed, 4 Jul 2018 at 19:25, Andrew Oberstar ***@***.***> wrote: Closed #90 <#90> via 0ff5198 <0ff5198> . — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#90 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABel_tylOg8fwaUy-J5rdY59MbiWn6Guks5uDQiigaJpZM4UpXKb> .