Skip to content

Conversation

@Uzlopak
Copy link
Contributor

@Uzlopak Uzlopak commented Sep 10, 2025

Currently benchmarks run twice if we collaborators create branches directly on the nodejs/undici repository. This fixes it by only running on our protected branches. Basically copy paste from the other workflows

This relates to...

Rationale

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

…es only

Currently benchmarks run twice if we collaborators create branches directly on the nodejs/undici repository. This fixes it by only running on our protected branches. Basically copy paste from the other workflows
@Uzlopak Uzlopak requested a review from Copilot September 10, 2025 23:11
Copy link
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 optimizes the benchmark CI workflow to prevent duplicate runs when collaborators push to branches on the main repository. The workflow now only runs on protected branches (main, current, next, v*) for push events while still running on all pull requests.

  • Changes trigger conditions from general push/pull_request to specific branch targeting
  • Maintains pull request benchmarking while reducing unnecessary runs on feature branches

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@Uzlopak Uzlopak merged commit 5c1b4a5 into main Sep 15, 2025
27 of 28 checks passed
@github-actions github-actions bot mentioned this pull request Jan 5, 2026
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