pseudo-nymous opened a new pull request, #14369: URL: https://github.com/apache/lucene/pull/14369
### Description This pull request contains a fix for changelog automation that has been added recently. We have seen failures where either diff calculation wrt base commit was wrong or base commit was not even available leading to fatal error. This happens because we are using `pull_request_target` event type which checks out `merge base commit` where as we want to checkout `head commit` to do proper diff calculation wrt changelog file. My initial testing was based on `pull_request` which worked as expected since it checkouts `merge head commit`. We tried a fix for this here: https://github.com/apache/lucene/pull/14355. But we didn't fetch the full history appropriately. This PR adds a fix for this by pulling all the associated history using `git pull --unshallow origin ${{ github.event.pull_request.head.ref }}`. Tested all the changes here: https://github.com/pseudo-nymous/lucene/pull/22 Also, added optimization with `ref: ${{ github.event.pull_request.head.sha }}`, where it will only pull the head commit instead of pulling all the references. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
