Skip to content

ci: add concurrency in workflows#4527

Open
Uzlopak wants to merge 2 commits into
mainfrom
add-concurrency
Open

ci: add concurrency in workflows#4527
Uzlopak wants to merge 2 commits into
mainfrom
add-concurrency

Conversation

@Uzlopak

@Uzlopak Uzlopak commented Sep 7, 2025

Copy link
Copy Markdown
Contributor

This relates to...

Rationale

Changes

Features

Bug Fixes

Breaking Changes and Deprecations

Status

@Uzlopak Uzlopak requested a review from Copilot September 7, 2025 22:23

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

This PR adds concurrency controls to GitHub Actions workflows to prevent multiple workflow runs from executing simultaneously for the same branch/PR, improving CI efficiency and resource usage.

  • Adds concurrency groups to CI and benchmark workflows with automatic cancellation of in-progress runs
  • Removes the "Test h2" step from the Node.js workflow
  • Configures workflows to cancel previous runs when new commits are pushed

Reviewed Changes

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

File Description
.github/workflows/ci.yml Adds concurrency group configuration for CI workflows
.github/workflows/bench.yml Adds concurrency group configuration for benchmark workflows
.github/workflows/nodejs.yml Removes the "Test h2" workflow step

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

Comment thread .github/workflows/nodejs.yml Outdated
id: test-h2
env:
CI: true
NODE_V8_COVERAGE: ${{ inputs.codecov == true && './coverage/tmp' || '' }}

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.

why this change?

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