Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🩹 fix coverage configuration to properly exclude the literals.py file #552

Merged
merged 2 commits into from
Feb 17, 2025

Conversation

burgholzer
Copy link
Member

Description

This pull request fixes the coverage configuration to properly exclude the literals.py file from coverage reports. This ensures accurate reporting by excluding files that are not relevant to coverage metrics.

Checklist:

  • The pull request only contains commits that are related to it.
  • I have added appropriate tests and documentation.
  • I have made sure that all CI jobs on GitHub pass.
  • The pull request introduces no new warnings and follows the project's style guidelines

@burgholzer burgholzer self-assigned this Feb 17, 2025
@burgholzer burgholzer added python Anything related to Python code continuous integration Anything related to the CI setup labels Feb 17, 2025
@burgholzer burgholzer enabled auto-merge (squash) February 17, 2025 16:38
@burgholzer burgholzer added this to the MQT QCEC v3.0 milestone Feb 17, 2025
@burgholzer burgholzer disabled auto-merge February 17, 2025 23:46
@burgholzer burgholzer merged commit 379c211 into main Feb 17, 2025
20 checks passed
@burgholzer burgholzer deleted the coverage-excludes branch February 17, 2025 23:46
Copy link

codecov bot commented Feb 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.8%. Comparing base (9da0e91) to head (dba0f0a).
Report is 5 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #552     +/-   ##
=======================================
+ Coverage   95.5%   95.8%   +0.2%     
=======================================
  Files         40      39      -1     
  Lines       1839    1834      -5     
  Branches     223     223             
=======================================
  Hits        1757    1757             
+ Misses        82      77      -5     
Flag Coverage Δ *Carryforward flag
cpp 96.0% <ø> (ø) Carriedforward from 9da0e91
python 94.9% <ø> (+1.1%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.

see 1 file with indirect coverage changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
continuous integration Anything related to the CI setup python Anything related to Python code
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant