Skip to content

Also run tests on commits to main#292

Merged
PGijsbers merged 2 commits intomainfrom
cov-on-main
Mar 25, 2026
Merged

Also run tests on commits to main#292
PGijsbers merged 2 commits intomainfrom
cov-on-main

Conversation

@PGijsbers
Copy link
Copy Markdown
Contributor

and upload coverage, as a base report is required to report of loss of code coverage

@PGijsbers PGijsbers added the automation CI/CD, pre-commit, ... label Mar 25, 2026
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@3dd3b33). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #292   +/-   ##
=======================================
  Coverage        ?   54.15%           
=======================================
  Files           ?       37           
  Lines           ?     1553           
  Branches        ?      135           
=======================================
  Hits            ?      841           
  Misses          ?      710           
  Partials        ?        2           

☔ View full report in Codecov by Sentry.
📢 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.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 25, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 8ffd35fb-aa2f-4d9e-bec7-e91cd6f427a7

📥 Commits

Reviewing files that changed from the base of the PR and between 3dd3b33 and 7f6e7e2.

📒 Files selected for processing (1)
  • .github/workflows/tests.yml

Walkthrough

The workflow configuration is modified to trigger on push events to the main branch alongside existing pull_request path filters. Additionally, the pytest concurrency argument is updated to conditionally use a value of 1 when mutations is true in the matrix, otherwise defaulting to auto. The test marker expression and overall job structure remain unchanged. Total modifications comprise 5 additions and 1 deletion.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly and accurately describes the main objective of the PR: enabling test execution on commits to the main branch.
Description check ✅ Passed The description relates to the changeset by explaining the purpose of running tests on main—uploading coverage reports as a base for detecting code coverage loss.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cov-on-main

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 and usage tips.

@PGijsbers PGijsbers merged commit fde90bf into main Mar 25, 2026
9 checks passed
@PGijsbers PGijsbers deleted the cov-on-main branch March 25, 2026 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation CI/CD, pre-commit, ...

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant