Skip to content

Add Timestampe git metadata file#306

Closed
ngehrsitz wants to merge 1 commit intoconcourse:masterfrom
ngehrsitz:master
Closed

Add Timestampe git metadata file#306
ngehrsitz wants to merge 1 commit intoconcourse:masterfrom
ngehrsitz:master

Conversation

@ngehrsitz
Copy link
Contributor

This commit adds generation of a timestamp metadata file that can be used to tag builds. This is useful when running parallel builds of a pipeline to determine which version is more up to date after pushing to a build storage like artifactory.

@ngehrsitz ngehrsitz force-pushed the master branch 2 times, most recently from cd55143 to 6cb8893 Compare February 19, 2020 11:07
This commit adds generation of a timestamp metadata file that can be used to tag builds. This is useful when running parallel builds of a pipeline to determine which version is more up to date after pushing to a build storage like artifactory.

Signed-off-by: ngrsdhbw <gehrsitz.norman@student.dhbw-karlsruhe.de>
Copy link
Contributor

@YoussB YoussB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feature seems like a really solid idea, thanks! Can we get a test for this? The other 'additional files populated' have tests here. I'm trying to ask for tests whenever possible to prevent regressions since this resource will (one day) get rewritten.

@YoussB
Copy link
Contributor

YoussB commented May 19, 2020

Hey @ngrsdhbw,
I am going to close this now for inactivity, I still believe this is a solid idea, would appreciate you reopening with the tests added whenever you want and I will be more than happy to merge it.

Thanks!

@YoussB YoussB closed this May 19, 2020
@ngehrsitz
Copy link
Contributor Author

I added the missing tests and a configuration option to choose the timestamp format on my branch git-timestamp. @YoussB could you reopen the pull request or should i make a new one?

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

Successfully merging this pull request may close these issues.

2 participants