Skip to content

Comments

chore: tophat builds for PR scope#1841

Open
maxy-shpfy wants to merge 1 commit intomasterfrom
02-22-chore_tophat_builds_for_pr_scope
Open

chore: tophat builds for PR scope#1841
maxy-shpfy wants to merge 1 commit intomasterfrom
02-22-chore_tophat_builds_for_pr_scope

Conversation

@maxy-shpfy
Copy link
Collaborator

@maxy-shpfy maxy-shpfy commented Feb 22, 2026

Description

Contributes to https://github.com/Shopify/oasis-frontend/issues/180

Added a GitHub workflow that automatically triggers Tophat builds in the oasis-frontend repository when pull requests are opened or updated. The workflow extracts PR information (branch, number, SHA), makes a Buildkite API call to trigger the build, and posts a comment on the PR with instructions on how to tophat the changes using a URL parameter.

Related Issue and Pull requests

Type of Change

  • Bug fix
  • New feature
  • Improvement
  • Cleanup/Refactor
  • Breaking change
  • Documentation update

Checklist

  • I have tested this does not break current pipelines / runs functionality
  • I have tested the changes on staging

Screenshots (if applicable)

Tophat Links Demo - v1.mp4 (uploaded via Graphite)

Test Instructions

  1. Create a test pull request to verify the workflow triggers
  2. Check that the workflow runs and successfully makes the Buildkite API call
  3. Verify that a comment is posted on the PR with tophat instructions
  4. Confirm the comment includes the correct branch name and SHA for the tophat URL parameter

Additional Comments

This workflow requires the BUILDKITE_BEARER, and BUILDKITE_PIPELINE secrets to be configured with appropriate permissions to trigger builds in Buildkite. The workflow runs on all file changes in pull requests and provides developers with an automated way to test their changes on the staging environment.

Copy link
Collaborator Author

maxy-shpfy commented Feb 22, 2026

@maxy-shpfy maxy-shpfy force-pushed the 02-22-chore_tophat_builds_for_pr_scope branch 2 times, most recently from bcd9eb8 to 13c42d7 Compare February 22, 2026 20:14
@github-actions
Copy link

github-actions bot commented Feb 22, 2026

🎩 To tophat this PR:

You can add the following URL parameter to your browser to tophat this PR:

`?tophat_location=02-22-chore_tophat_builds_for_pr_scope/2d2a9c5`

@maxy-shpfy maxy-shpfy changed the base branch from master to graphite-base/1841 February 24, 2026 00:54
@maxy-shpfy maxy-shpfy force-pushed the 02-22-chore_tophat_builds_for_pr_scope branch from 13c42d7 to 42714f4 Compare February 24, 2026 00:54
@maxy-shpfy maxy-shpfy changed the base branch from graphite-base/1841 to 02-23-poc_support_tophat_loader_in_index.html_for_staging February 24, 2026 00:54
@maxy-shpfy maxy-shpfy force-pushed the 02-22-chore_tophat_builds_for_pr_scope branch from 42714f4 to e7a113b Compare February 24, 2026 03:46
@maxy-shpfy maxy-shpfy mentioned this pull request Feb 24, 2026
8 tasks
@maxy-shpfy maxy-shpfy force-pushed the 02-22-chore_tophat_builds_for_pr_scope branch 5 times, most recently from e0c8ad9 to eef7699 Compare February 24, 2026 20:44
@maxy-shpfy maxy-shpfy marked this pull request as ready for review February 24, 2026 21:12
@maxy-shpfy maxy-shpfy requested a review from a team as a code owner February 24, 2026 21:12
@maxy-shpfy maxy-shpfy force-pushed the 02-22-chore_tophat_builds_for_pr_scope branch from eef7699 to 16c0ebf Compare February 24, 2026 21:35
@maxy-shpfy maxy-shpfy changed the base branch from 02-23-poc_support_tophat_loader_in_index.html_for_staging to graphite-base/1841 February 24, 2026 22:49
@maxy-shpfy maxy-shpfy force-pushed the 02-22-chore_tophat_builds_for_pr_scope branch from 16c0ebf to f06d71f Compare February 24, 2026 22:54
@graphite-app graphite-app bot changed the base branch from graphite-base/1841 to master February 24, 2026 22:54
@maxy-shpfy maxy-shpfy force-pushed the 02-22-chore_tophat_builds_for_pr_scope branch from f06d71f to 2d2a9c5 Compare February 24, 2026 22:54
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.

3 participants