Skip to content

Separate the concerns: dependabot and manual PR labelling for testing#3527

Merged
zinduolis merged 2 commits intomasterfrom
red/fix_dependabot_automerge
Mar 11, 2026
Merged

Separate the concerns: dependabot and manual PR labelling for testing#3527
zinduolis merged 2 commits intomasterfrom
red/fix_dependabot_automerge

Conversation

@zinduolis
Copy link
Contributor

Category

Core Functionality

Feature/Issue Description

Q: Please give a brief summary of your feature/fix
A: Fixes dependabot auto-merge by running BrowserStack tests automatically for dependabot PRs, without bypassing branch protection for other PRs.

Q: Give a technical rundown of what you have changed (if applicable)
A:

  • Split into two workflows to avoid GitHub's "skipped job = passed check" branch protection bypass
  • github_actions.yml: reverted to labeled-only trigger, added concurrency group, added try/catch on label removal
  • dependabot_auto_label.yml (new): on opened/synchronize, auto-applies safe_to_test label for dependabot[bot] PRs only, which triggers the BrowserStack workflow via the labeled event

Test Cases

Q: Describe your test cases, what you have covered and if there are any use cases that still need addressing.
A:

  • Dependabot PRs: auto-labeled → BrowserStack runs → auto-merge proceeds
  • Fork PRs: no auto-label → required check stays pending → merge blocked until maintainer labels
  • Non-fork PRs: same as fork — no BrowserStack check created without label → merge blocked
  • Dependabot push updates: remove+re-add label ensures a fresh labeled event fires each time
  • Concurrent runs: concurrency with cancel-in-progress prevents duplicate BrowserStack runs

Wiki Page

N/A

zinduolis and others added 2 commits March 11, 2026 19:41
…on labeled, and create a lightweight auto-labeling workflow that applies safe_to_test to dependabot PRs
@zinduolis zinduolis added the safe_to_test Label to trigger tests on PR label Mar 11, 2026
@github-actions github-actions bot removed the safe_to_test Label to trigger tests on PR label Mar 11, 2026
@zinduolis zinduolis merged commit 5252c04 into master Mar 11, 2026
6 checks passed
@zinduolis zinduolis deleted the red/fix_dependabot_automerge branch March 11, 2026 09:49
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.

1 participant