Skip to content

Point integration testing guidance to pack workflow#8190

Merged
primer[bot] merged 2 commits into
mainfrom
copilot/remove-canary-testing-mentions
Jul 20, 2026
Merged

Point integration testing guidance to pack workflow#8190
primer[bot] merged 2 commits into
mainfrom
copilot/remove-canary-testing-mentions

Conversation

Copilot AI commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Contributor guidance referenced deprecated canary and fallback testing flows. This updates all contributor-facing instructions to use the pack testing workflow.

Changelog

New

None.

Changed

  • Direct integration testing guidance to the pack testing workflow.

Removed

  • Canary release testing guidance.
  • Fallback workflow guidance.

Rollout strategy

  • Patch release
  • Minor release
  • Major release; if selected, include a written rollout or migration plan
  • None; repository guidance only.

Testing & Reviewing

  • Review the pull request template and generated integration-test comments for consistent workflow guidance.

Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>
@changeset-bot

changeset-bot Bot commented Jul 17, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 10113b5

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@joshblack joshblack left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@copilot you can keep referring to it as the integration workflow. pack is an implementation detail and we don't want to communicate that externally

Co-authored-by: joshblack <3901764+joshblack@users.noreply.github.com>

Copilot AI commented Jul 17, 2026

Copy link
Copy Markdown
Contributor Author

@copilot you can keep referring to it as the integration workflow. pack is an implementation detail and we don't want to communicate that externally

Updated the guidance to use “integration workflow” throughout in 10113b5.

Copilot AI requested a review from joshblack July 17, 2026 18:39
@joshblack
joshblack marked this pull request as ready for review July 17, 2026 18:39
@joshblack
joshblack requested a review from a team as a code owner July 17, 2026 18:39
@joshblack
joshblack requested review from Copilot and jonrohan July 17, 2026 18:39

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

Updates contributor-facing guidance to remove deprecated canary/fallback integration-testing instructions and instead direct GitHub staff to the github/github-ui integration workflow.

Changes:

  • Simplifies the automated “recommend integration tests” PR comment to remove fallback workflow and canary-release instructions.
  • Updates the PR template “Testing & Reviewing” guidance to point to the github/github-ui integration workflow link.
Show a summary per file
File Description
.github/workflows/recommend-integration-tests.yml Removes fallback/canary guidance from the bot-authored PR comment, keeping only the integration workflow + docs + skip label.
.github/pull_request_template.md Replaces canary-release label guidance with a direct link to the integration workflow for testing with github/github-ui.

Review details

  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Low

@joshblack
joshblack enabled auto-merge July 20, 2026 16:08
@primer
primer Bot disabled auto-merge July 20, 2026 19:40
@primer
primer Bot added this pull request to the merge queue Jul 20, 2026
Merged via the queue into main with commit fbf2fbd Jul 20, 2026
68 of 69 checks passed
@primer
primer Bot deleted the copilot/remove-canary-testing-mentions branch July 20, 2026 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration-tests: skipped manually Changes in this PR do not require an integration test skip changeset This change does not need a changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants