search

peternied / bake-time

GitHub Action to ensure pull requests bake before merging
Apache License 2.0
0 stars 1 forks source link

Fix incorrect timestamp evaluation with `ignore-non-push-updates` option #9

Closed ngehrsitz closed 9 months ago

ngehrsitz commented 9 months ago

This fixes the ignore-non-push-updates option introduced in #8. I did not find another way to get the push date of commits other than to look at the PushEvents for the repo since pushedDate from GraphQL has been removed from the API: https://github.com/orgs/community/discussions/24730#discussioncomment-3245265 https://docs.github.com/en/graphql/reference/objects#commit Demo on https://github.com/ngehrsitz/bake-time-test/pull/4 and https://github.com/ngehrsitz/bake-time-test/pull/6

peternied commented 9 months ago

Thanks @ngehrsitz, released in v3.3