Skip to content

Account for tiny time differences between commits#113

Merged
mcmire merged 1 commit intomainfrom
get-most-recent-tag-with-tiny-differences
Jun 14, 2022
Merged

Account for tiny time differences between commits#113
mcmire merged 1 commit intomainfrom
get-most-recent-tag-with-tiny-differences

Conversation

@mcmire
Copy link
Copy Markdown
Contributor

@mcmire mcmire commented Jun 1, 2022

When two tags are created in a repo that differ by mere milliseconds or
smaller, such as would happen in an automated high-level test of the
create-release-pr action, git rev-list --tags seems to get confused,
and so getMostRecentTag may not always return the latest tag. Adding
--date-order seems to fix this.

When two tags are created in a repo that differ by mere milliseconds or
smaller, such as would happen in an automated high-level test of the
`create-release-pr` action, `git rev-list --tags` seems to get confused,
and so `getMostRecentTag` may not always return the latest tag. Adding
`--date-order` seems to fix this.
@mcmire mcmire requested a review from a team as a code owner June 1, 2022 20:21
Copy link
Copy Markdown
Member

@rekmarks rekmarks left a comment

Choose a reason for hiding this comment

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

LGTM!

@mcmire mcmire merged commit d0719d1 into main Jun 14, 2022
@mcmire mcmire deleted the get-most-recent-tag-with-tiny-differences branch June 14, 2022 22:07
@mcmire mcmire mentioned this pull request Jun 15, 2022
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