Skip to content

feat: forward GITHUB_BASE_REF and GITHUB_BASE_SHA to benchmark pipelines#237

Closed
wdconinc wants to merge 2 commits intomasterfrom
base
Closed

feat: forward GITHUB_BASE_REF and GITHUB_BASE_SHA to benchmark pipelines#237
wdconinc wants to merge 2 commits intomasterfrom
base

Conversation

@wdconinc
Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.

Pass the PR target branch name and SHA through the GitLab trigger variables so downstream benchmark pipelines can use them for artifact comparison.

What is the urgency of this PR?

  • High
  • Medium
  • Low

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Optimization (issue #__)
  • Updated documentation
  • other: __

Please check if any of the following apply

  • This PR introduces breaking changes. Please describe changes users need to make below.
  • This PR changes default behavior. Please describe changes below.
  • AI was used in preparing this PR. Please describe usage below.

AI write this. Well, not this.

Pass the PR target branch name and SHA through the GitLab trigger variables
so downstream benchmark pipelines can use them for artifact comparison.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 15, 2026 00:51
Copy link
Copy Markdown
Contributor

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

This PR forwards the PR target branch name and target SHA from GitHub Actions into the GitLab-triggered benchmark pipelines so downstream benchmarks can compare artifacts against the correct base.

Changes:

  • Add GITHUB_BASE_REF and GITHUB_BASE_SHA to the variables passed from GitHub Actions to the GitLab trigger.
  • Propagate these variables into the .benchmarks job template so they’re available to downstream triggered benchmark pipelines.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/mirror.yaml Adds GITHUB_BASE_REF/GITHUB_BASE_SHA to the GitLab trigger variables computed from the PR payload.
.gitlab-ci.yml Forwards GITHUB_BASE_REF/GITHUB_BASE_SHA through the .benchmarks template variables.

Comment thread .github/workflows/mirror.yaml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@wdconinc
Copy link
Copy Markdown
Contributor Author

Implemented capybara differently in #253, #254, #255.

@wdconinc wdconinc closed this Apr 24, 2026
@wdconinc wdconinc deleted the base branch April 24, 2026 16:15
@wdconinc wdconinc restored the base branch April 24, 2026 17:55
@wdconinc wdconinc deleted the base branch April 24, 2026 18:16
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.

2 participants