Skip to content

Keep every main run instead of dropping pending ones - #48

Merged
LucaCappelletti94 merged 1 commit into
mainfrom
upstream/main-ci-concurrency
Oct 30, 2026
Merged

LucaCappelletti94 merged 1 commit into
mainfrom
upstream/main-ci-concurrency

Conversation

@LucaCappelletti94

Copy link
Copy Markdown
Owner

CI, CodeQL and Coverage put every run on main in one concurrency group without cancelling, and GitHub keeps only one pending run per group. When two merges land while a run is in progress, the earlier one is dropped before it starts and that commit never gets a verdict on main. sqlcipher-src lost the CI runs of two merged pull requests this way.

A run on main now gets a group keyed by its run id, so each landed commit keeps its run. Pull request branches still share one group per branch, so a new push still cancels the run it supersedes.

@LucaCappelletti94 LucaCappelletti94 added the github_actions Pull requests that update GitHub Actions code label Oct 30, 2026
@coderabbitai

coderabbitai Bot commented Oct 30, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Central YAML; Path: .coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: d4ba0246-82a1-43f0-9db0-1d0767c6bc15
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (b7651db) to head (9f32700).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #48   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            3         3           
  Branches         3         3           
=========================================
  Hits             3         3           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LucaCappelletti94
LucaCappelletti94 merged commit 2ce0c8d into main Oct 30, 2026
23 checks passed
@LucaCappelletti94
LucaCappelletti94 deleted the upstream/main-ci-concurrency branch October 30, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant