Skip to content

docs: Warn that scopes action requires a pull_request context#10431

Closed
remyduthu wants to merge 1 commit intomainfrom
devs/remyduthu/docs/scopes-requires-pr-context/I2c2bf6ceef295ac071d2108525107432f84ec854
Closed

docs: Warn that scopes action requires a pull_request context#10431
remyduthu wants to merge 1 commit intomainfrom
devs/remyduthu/docs/scopes-requires-pr-context/I2c2bf6ceef295ac071d2108525107432f84ec854

Conversation

@remyduthu
Copy link
Contributor

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Copilot AI review requested due to automatic review settings March 6, 2026 10:21
@mergify mergify bot had a problem deploying to Mergify Merge Protections March 6, 2026 10:21 Failure
@mergify
Copy link
Contributor

mergify bot commented Mar 6, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🔴 👀 Review Requirements

This rule is failing.
  • any of:
    • #approved-reviews-by >= 2
    • author = dependabot[bot]
    • author = mergify-ci-bot

🔴 🔎 Reviews

This rule is failing.
  • #review-requested = 0
  • #review-threads-unresolved = 0
  • #changes-requested-reviews-by = 0

🟠 🤖 Continuous Integration

Waiting checks: test.
  • all of:
    • check-success = test
    • check-success = build
    • check-success = lint
    • any of:
      • check-success = test-broken-links
      • label = ignore-broken-links
    • any of:
      • check-success=Cloudflare Pages
      • -head-repo-full-name~=^Mergifyio/

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

jd
jd previously approved these changes Mar 6, 2026
@remyduthu remyduthu marked this pull request as ready for review March 6, 2026 10:23
@mergify mergify bot requested a review from a team March 6, 2026 10:24
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a documentation caution to prevent users from running the scopes action outside of a pull request context, which would otherwise cause GitHub Actions workflow failures.

Changes:

  • Document that the scopes action requires a pull request context.
  • Provide a suggested if: guard for workflows that also run on push.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Change-Id: I2c2bf6ceef295ac071d2108525107432f84ec854
@remyduthu remyduthu force-pushed the devs/remyduthu/docs/scopes-requires-pr-context/I2c2bf6ceef295ac071d2108525107432f84ec854 branch from 7e97408 to 56d0f87 Compare March 6, 2026 10:28
@mergify mergify bot dismissed jd’s stale review March 6, 2026 10:29

Pull request has been modified.

@mergify mergify bot had a problem deploying to Mergify Merge Protections March 6, 2026 10:29 Failure
@remyduthu remyduthu closed this Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants