ci: port PR template, ready-for-review label + PR-review gate workflows from ruby-sdk#57
Merged
Conversation
…ws from browserstack-ruby-sdk - Add .github/PULL_REQUEST_TEMPLATE.md (release-section gate contract) - Add .github/workflows/ready-for-review-label.yml - Add .github/workflows/sdk-pr-review-gate.yml Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
kamal-kaur04
approved these changes
Jul 17, 2026
Bhargavi-BS
approved these changes
Jul 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is this about?
Ports the PR-hygiene tooling from
browserstack-ruby-sdkinto this repo, verbatim:.github/PULL_REQUEST_TEMPLATE.md## Releasesection (version bump + internal release notes) that theready-for-reviewgate reads.github/workflows/ready-for-review-label.yml## Releasesection, removes theready-for-reviewlabel when the body fails the gate, and fails the check until the label is present. Manually-applied, required merge gate..github/workflows/sdk-pr-review-gate.ymlSDK PR Review Gate) — green only when theready-for-reviewlabel is present and the SDK PR Review Agent's latest verdict marker reportsstate=successon the current head SHA.Source:
browserstack-ruby-sdk@23b89cb. Files copied unchanged so behaviour stays identical across SDK repos.Related Jira task/s
Release (mandatory for every PR — required for the
ready-for-reviewlabel)Version bump: (required — tick exactly one)
Release notes type: (optional)
Release notes (customer-facing): (optional but encouraged)
Release notes (internal): (required — engineer-facing; what actually changed / why)
ready-for-reviewlabel workflow, and SDK PR Review gate workflow frombrowserstack-ruby-sdkso PR hygiene is consistent across SDK repos.Operational prerequisites (before these gates function)
These files are copied as-is; to make them enforce anything the repo admin still needs to:
ready-for-reviewlabel in the repo (the workflow only ever removes it — the author applies it manually; it does not create it).SLACK_PR_SLA_WEBHOOKsecret — without it, Slack notify steps log and skip (non-fatal).<!-- sdk-pr-review:verdict sha=… state=… -->marker; until thenSDK PR Review Gatestays red on real PRs.check-readyandgatechecks (plus "Require approvals") once the above are in place.Until branch protection requires them, both new workflows run as informational (non-blocking) checks. This PR's own body follows the new template so the release-gate logic passes; the
ready-for-reviewlabel check will still show red here until the label is applied — expected.Checklist
PR Validations
Run Tests: Comment RUN_TESTS to trigger sanity tests.
🤖 Generated with Claude Code