Skip to content

Move gcovr options to config file#21663

Merged
iluuu1994 merged 1 commit intophp:masterfrom
iluuu1994:gcovr-config
Apr 7, 2026
Merged

Move gcovr options to config file#21663
iluuu1994 merged 1 commit intophp:masterfrom
iluuu1994:gcovr-config

Conversation

@iluuu1994
Copy link
Copy Markdown
Member

Having these in the make file is very annoying, because it requires rebuilding from scratch and rerunning tests just to see if some flag is effective.

Note: gcovr config files are marked as experiemental:

https://gcovr.com/en/latest/guide/configuration.html

However, they've been experimental for ~4 years and it's unlikely it'll go anywhere. Even if, we use this for internal purposes only.

@iluuu1994 iluuu1994 changed the title More gcovr options to config file Move gcovr options to config file Apr 7, 2026
Having these in the make file is very annoying, because it requires rebuilding
from scratch and rerunning tests just to see if some flag is effective.
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.77%. Comparing base (6d62d87) to head (13621a7).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #21663   +/-   ##
=======================================
  Coverage   70.77%   70.77%           
=======================================
  Files         812      812           
  Lines      296976   296976           
  Branches    84534    84534           
=======================================
  Hits       210199   210199           
+ Misses      56610    56608    -2     
- Partials    30167    30169    +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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@iluuu1994 iluuu1994 merged commit 5bdfb1c into php:master Apr 7, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant