Skip to content

Avoid gh pr checkout in pull request helper workflow#4711

Merged
xiang17 merged 1 commit into
microsoft:mainfrom
xiang17:xiang17/pull-request-helper-head-checkout
May 22, 2026
Merged

Avoid gh pr checkout in pull request helper workflow#4711
xiang17 merged 1 commit into
microsoft:mainfrom
xiang17:xiang17/pull-request-helper-head-checkout

Conversation

@xiang17
Copy link
Copy Markdown
Member

@xiang17 xiang17 commented May 22, 2026

Avoid gh pr checkout in Dependabot helper workflow by checking out PR branch directly

This will be done in multiple steps:

  1. [This PR] Merge a small workflow PR to main that checks out the Dependabot PR branch directly instead of running gh pr checkout.
  2. Rerun or refresh the affected Dependabot PRs (Bump gradle-wrapper from 9.4.1 to 9.5.1 in /perf-tests #4702) so their pull_request_target jobs use the updated workflow from main.
  3. In a follow-up cleanup PR, normalize Gradle wrapper .bat line endings so Windows checkouts and GitHub Actions agree.
  4. After both changes land, future Dependabot wrapper PRs should avoid the intermittent checkout conflict on gradlew.bat files.

Issue

Today there has been two PRs stuck on the "Pull request helper" step due to failure: #4687, #4702:

Run gh pr checkout $NUMBER
From https://github.com/microsoft/ApplicationInsights-Java
 * [new branch]          dependabot/gradle/otelInstrumentationAlphaVersion-2.27.0-alpha -> origin/dependabot/gradle/otelInstrumentationAlphaVersion-2.27.0-alpha
error: Your local changes to the following files would be overwritten by checkout:
	gradlew.bat
	perf-tests/gradlew.bat
Please commit your changes or stash them before you switch branches.
Aborting
failed to run git: exit status 1
Error: Process completed with exit code 1.

@xiang17 xiang17 merged commit 98e2410 into microsoft:main May 22, 2026
285 of 290 checks passed
@xiang17 xiang17 deleted the xiang17/pull-request-helper-head-checkout branch May 22, 2026 19:42
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