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

✨ Switch to mqt-core Python package #432

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Conversation

burgholzer
Copy link
Member

Description

This is an alternative to #355 and hopefully marks the final try of directly incorporating the mqt-core Python package. See #355 and #352 for some history on this topic.

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 added feature New feature or request dependencies Pull requests that update a dependency file python Anything related to Python code packaging Anything related to Python packaging c++ Anything related to C++ code minor Part of a minor release labels Aug 6, 2024
@burgholzer burgholzer self-assigned this Aug 6, 2024
Copy link

codecov bot commented Aug 6, 2024

Codecov Report

Attention: Patch coverage is 95.50562% with 4 lines in your changes missing coverage. Please review.

Project coverage is 95.5%. Comparing base (5ba703b) to head (e22ad39).
Report is 15 commits behind head on main.

Files with missing lines Patch % Lines
src/mqt/qcec/__init__.py 88.8% 2 Missing ⚠️
src/mqt/qcec/parameterized.py 96.8% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main    #432   +/-   ##
=====================================
  Coverage   95.4%   95.5%           
=====================================
  Files         40      40           
  Lines       1834    1846   +12     
  Branches     223     225    +2     
=====================================
+ Hits        1751    1763   +12     
  Misses        83      83           
Flag Coverage Δ
cpp 96.1% <ø> (+0.1%) ⬆️
python 93.4% <95.5%> (-0.4%) ⬇️
Files with missing lines Coverage Δ
src/mqt/qcec/verify.py 100.0% <100.0%> (ø)
src/mqt/qcec/verify_compilation_flow.py 100.0% <ø> (ø)
src/mqt/qcec/__init__.py 91.3% <88.8%> (-8.7%) ⬇️
src/mqt/qcec/parameterized.py 94.5% <96.8%> (-0.4%) ⬇️

... and 1 file with indirect coverage changes

@burgholzer burgholzer force-pushed the use-mqt-core-package branch 6 times, most recently from 96ff41e to db6cb20 Compare August 10, 2024 13:34
@burgholzer burgholzer force-pushed the use-mqt-core-package branch 5 times, most recently from 055c342 to bf94b85 Compare August 14, 2024 17:25
@burgholzer burgholzer force-pushed the use-mqt-core-package branch 2 times, most recently from 77145ad to 64dfa87 Compare September 4, 2024 21:26
@burgholzer burgholzer added major Part of a major release and removed minor Part of a minor release labels Sep 4, 2024
burgholzer added a commit that referenced this pull request Sep 29, 2024
## Description

This PR updates the CD workflow so that anytime it runs on a push to
`main`, it uploads the resulting package to Test PyPI. This allows to
battle test the packages before official releases.
This will be especially important in the context of #432
## Checklist:

<!---
This checklist serves as a reminder of a couple of things that ensure
your pull request will be merged swiftly.
-->

- [x] The pull request only contains commits that are related to it.
- [x] I have added appropriate tests and documentation.
- [x] I have made sure that all CI jobs on GitHub pass.
- [x] The pull request introduces no new warnings and follows the
project's style guidelines.
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
(temporarily as a git dependency)

Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
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 feature New feature or request major Part of a major release packaging Anything related to Python packaging python Anything related to Python code
Projects
Status: In Progress
Status: In Progress
Development

Successfully merging this pull request may close these issues.

1 participant