Skip to content

chore: Require acctest label for prs#3540

Merged
deiga merged 1 commit into
integrations:mainfrom
stevehipwell:require-acctest-label-for-prs
Jul 14, 2026
Merged

chore: Require acctest label for prs#3540
deiga merged 1 commit into
integrations:mainfrom
stevehipwell:require-acctest-label-for-prs

Conversation

@stevehipwell

Copy link
Copy Markdown
Collaborator

Resolves #ISSUE_NUMBER

This PR stops unnecessary acceptance test runs using up the API limit.


Before the change?

  • All non-fork PRs trigger acceptance tests

After the change?

  • Non-fork PRs require the acctest label to be set

Pull request checklist

  • Schema migrations have been created if needed (example)
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

Signed-off-by: Steve Hipwell <steve.hipwell@gmail.com>
@stevehipwell stevehipwell added this to the v6.14.0 milestone Jul 14, 2026
@stevehipwell stevehipwell requested review from Copilot and deiga July 14, 2026 15:16
@stevehipwell stevehipwell self-assigned this Jul 14, 2026
@stevehipwell stevehipwell added the Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR label Jul 14, 2026
@github-actions

Copy link
Copy Markdown

👋 Hi, and thank you for this contribution!

This repo is maintained by GitHub and community members on a best-effort basis. We'll get to this as soon as we can.

You can help us prioritize by joining the discussion on open issues and PRs, sharing details on the changes you need, and reviewing other contributions.


🤖 This is an automated message.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

These provider review instructions are being used. This PR adjusts the Acceptance Tests GitHub Actions workflow to avoid running acceptance tests on every same-repo PR update by requiring an explicit acctest label before tests will run in PR contexts, reducing unnecessary GitHub API usage.

Changes:

  • Gate acceptance test execution for pull_request_target events on whether the acctest label is present.
  • Simplify the PR fork vs. base-repo branching so the label gate happens once up-front.

@deiga deiga merged commit d6cc14c into integrations:main Jul 14, 2026
13 of 16 checks passed
@stevehipwell stevehipwell deleted the require-acctest-label-for-prs branch July 15, 2026 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Maintenance Any dependency, housekeeping, and clean up Issue or PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants