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

⬆️👽 Update mqt-core to its latest version #539

Closed
wants to merge 4 commits into from

Conversation

burgholzer
Copy link
Member

Description

This PR updates mqt-core to its latest version, which (amongst others) contains the changes from cda-tum/mqt-core#807.

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.

Signed-off-by: burgholzer <burgholzer@me.com>
…it difference

Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
@burgholzer burgholzer added dependencies Pull requests that update a dependency file c++ Anything related to C++ code labels Jan 21, 2025
@burgholzer burgholzer self-assigned this Jan 21, 2025
@burgholzer burgholzer enabled auto-merge January 21, 2025 20:24
Copy link

codecov bot commented Jan 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.5%. Comparing base (09de617) to head (a49cf1f).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #539   +/-   ##
=====================================
  Coverage   95.5%   95.5%           
=====================================
  Files         40      40           
  Lines       1840    1842    +2     
  Branches     223     224    +1     
=====================================
+ Hits        1758    1760    +2     
  Misses        82      82           
Flag Coverage Δ *Carryforward flag
cpp 96.0% <100.0%> (+<0.1%) ⬆️
python 93.7% <ø> (ø) Carriedforward from 09de617

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

Files with missing lines Coverage Δ
src/EquivalenceCheckingManager.cpp 92.9% <100.0%> (+<0.1%) ⬆️

@burgholzer
Copy link
Member Author

This is somewhat blocked on #432 due to the MQT::CorePython target being removed.
I will probably just cherry-pick these commits to the other PR.

@burgholzer
Copy link
Member Author

Closing in favor of #432.

@burgholzer burgholzer closed this Jan 22, 2025
auto-merge was automatically disabled January 22, 2025 10:43

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Anything related to C++ code dependencies Pull requests that update a dependency file
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant