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
When using actions/checkout@v4 with a config like the following:
- name: Checkout the code
uses: actions/checkout@v4
with:
fetch-depth: 0
The initial commit the repo is checked out at corresponds to pull/<pr-no>/merge at the time of the PR synchronization event, as expected. However, if the CI job is manually restarted after another PR is merged to the main branch, the SHA of the commit remains the same as it was prior to main being updated
With this change, restarts of the CI run will always fetch the latest pull/<pr-no>/merge
The impact of this is that when CI is broken on mainline and a fix is merged, PR branches must be updated to trigger a synchronization event (unless the workaround described is used)
Is this expected behavior? IE is the commit used not suppose to update on manual restarts of CI jobs?
The text was updated successfully, but these errors were encountered:
When using
actions/checkout@v4
with a config like the following:The initial commit the repo is checked out at corresponds to
pull/<pr-no>/merge
at the time of the PR synchronization event, as expected. However, if the CI job is manually restarted after another PR is merged to the main branch, the SHA of the commit remains the same as it was prior to main being updatedFor an example see: tenstorrent/tt-zephyr-platforms#119
This can be worked around by setting
ref: ${{ github.ref }}
in the action YAML, like so:With this change, restarts of the CI run will always fetch the latest
pull/<pr-no>/merge
The impact of this is that when CI is broken on mainline and a fix is merged, PR branches must be updated to trigger a synchronization event (unless the workaround described is used)
Is this expected behavior? IE is the commit used not suppose to update on manual restarts of CI jobs?
The text was updated successfully, but these errors were encountered: