FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

ci: run ci for clang format commit and skip workflow if newer is pushed by Miou-zora · Pull Request #418 · EngineSquared/EngineSquared · GitHub

ci: run ci for clang format commit and skip workflow if newer is pushed - #418

Merged
ripel2 merged 30 commits into
mainfrom
allows-gh-bot-to-run-ci
Jan 10, 2026
Merged

ci: run ci for clang format commit and skip workflow if newer is pushed#418
ripel2 merged 30 commits into
mainfrom
allows-gh-bot-to-run-ci

Conversation

Miou-zora commented Jan 9, 2026
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

Related to no issues.

To fix current CI that often fail with various reasons, I decided to:

  • allows GitHub bot to run other jobs of the CI
  • cancel previous jobs if a new one appears
  • disallows GitHub bot to run clang-format job (push)

Summary by CodeRabbit

  • Chores
    • Improved CI workflow reliability and optimized test coverage configuration.

✏️ Tip: You can customize this high-level summary in your review settings.

coderabbitai Bot commented Jan 9, 2026
edited
Loading

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

CI workflow configuration updated to unconditionally cancel in-progress jobs, skip linting for bot-generated commits via conditional detection, remove coverage-related artifacts before testing on Linux, and a new gcovr configuration file is introduced for code coverage settings.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/ci.yml
Modified to unconditionally cancel in-progress jobs. Added bot-detection step to conditionally skip linting and related operations when commit author is github-actions[bot]. Introduced Linux-specific cleanup step to remove coverage artifacts (*.gcno, *.gcda, *.gcov) before and during test execution. Ensures test coverage check runs after Linux tests.
Coverage Configuration
gcovr.cfg
New configuration file with single setting: merge-mode-functions=merge-use-line-0 for gcovr code coverage reporting.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A rabbit hops through workflows clean,
Bot detection steps machine!
Coverage files swept away with care,
CI runs now, quick and fair.

🚥 Pre-merge checks | ✅ 2 | ❌ 1 ❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions running CI for clang format commits and skipping workflows, but the PR's primary objective is to allow the GitHub Actions bot to run CI jobs by adding authentication. The title doesn't reflect this main goal. Consider revising the title to better reflect the primary change: allowing GitHub Actions bot to run CI workflows with proper authentication.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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

Miou-zora changed the title CI: add token to github action bot ci: add token to github action bot Jan 9, 2026
Comment thread .github/workflows/ci.yml Fixed
Miou-zora marked this pull request as ready for review January 9, 2026 17:15
Miou-zora requested review from a team and Copilot and removed request for Copilot January 9, 2026 17:15
Miou-zora changed the title ci: add token to github action bot ci: run ci for clang format commit Jan 9, 2026
Miou-zora changed the title ci: run ci for clang format commit ci: run ci for clang format commit and skip workflow if newer is pushed Jan 9, 2026
Miou-zora self-assigned this Jan 9, 2026
Miou-zora added the ci Everything related to continous integration label Jan 9, 2026

Copy link
Copy Markdown

ripel2 enabled auto-merge (squash) January 10, 2026 17:06
ripel2 merged commit 2e08ffc into main Jan 10, 2026
14 checks passed
ripel2 deleted the allows-gh-bot-to-run-ci branch January 10, 2026 17:11
coderabbitai Bot mentioned this pull request Mar 23, 2026
20 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Everything related to continous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL