You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, when using <useGitCommitTimestamp> to add the commit timestamp to the generated version, the timestamp is added in the current locale of the machine where the build is done.
This is a problem because the version becomes different for different developers. Everyone must be able to agree on which version a certain commit has.
Proposals
Suggesting to add another configuration to set which time zone to use, which can be set to the current time zone by default to keep backwards compatibility. Since we live in a global world where many companies and teams are spread across time zones it would make more sense to use UTC by default, but this I guess would be a breaking change.
Another idea would be to configure timestamps as <useGitCommitTimestamp>UTC</useGitCommitTimestamp>, not to have to introduce another configuration. Passing true would give the current behaviour of local time zone.
The text was updated successfully, but these errors were encountered:
Issue
version: 1.9.0
usage context:
Problem description:
Currently, when using
<useGitCommitTimestamp>
to add the commit timestamp to the generated version, the timestamp is added in the current locale of the machine where the build is done.This is a problem because the version becomes different for different developers. Everyone must be able to agree on which version a certain commit has.
Proposals
Suggesting to add another configuration to set which time zone to use, which can be set to the current time zone by default to keep backwards compatibility. Since we live in a global world where many companies and teams are spread across time zones it would make more sense to use UTC by default, but this I guess would be a breaking change.
Another idea would be to configure timestamps as
<useGitCommitTimestamp>UTC</useGitCommitTimestamp>
, not to have to introduce another configuration. Passingtrue
would give the current behaviour of local time zone.The text was updated successfully, but these errors were encountered: