Skip to content

Conversation

@burgholzer
Copy link
Member

Description

Pulled out from #881.
Adds a convenience macro for adding MQT Core libraries and corresponding aliases.

Checklist:

  • The pull request only contains commits that are focused and relevant to this change.
  • I have added appropriate tests that cover the new/changed functionality.
  • I have updated the documentation to reflect these changes.
  • I have added entries to the changelog for any noteworthy additions, changes, fixes or removals.
  • I have added migration instructions to the upgrade guide (if needed).
  • The changes follow the project's style guidelines and introduce no new warnings.
  • The changes are fully tested and pass the CI checks.
  • I have reviewed my own code changes.

ystade and others added 2 commits May 22, 2025 14:31
Signed-off-by: burgholzer <burgholzer@me.com>
Signed-off-by: burgholzer <burgholzer@me.com>
@burgholzer burgholzer self-assigned this May 22, 2025
@burgholzer burgholzer added the usability Anything related to usability label May 22, 2025
@burgholzer burgholzer added the c++ Anything related to C++ code label May 22, 2025
@burgholzer burgholzer moved this to In Progress in MQT Core May 22, 2025
@burgholzer burgholzer enabled auto-merge (squash) May 22, 2025 12:47
@burgholzer burgholzer disabled auto-merge May 22, 2025 12:54
@burgholzer burgholzer enabled auto-merge (squash) May 22, 2025 12:58
@burgholzer burgholzer merged commit d2dca9b into main May 22, 2025
30 checks passed
@burgholzer burgholzer deleted the mqt-core-lib-cmake-utility branch May 22, 2025 13:28
@github-project-automation github-project-automation bot moved this from In Progress to Done in MQT Core May 22, 2025
burgholzer added a commit that referenced this pull request May 22, 2025
…#962)

## Description

Also pulled out from #881.
Builds on #961 and adds a dedicated option for only building the MQT
Core libraries as shared libraries without forcing this on every other
library.

## 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 focused and
relevant to this change.
- [x] I have added appropriate tests that cover the new/changed
functionality.
- [x] I have updated the documentation to reflect these changes.
- [x] I have added entries to the changelog for any noteworthy
additions, changes, fixes or removals.
- [x] I have added migration instructions to the upgrade guide (if
needed).
- [x] The changes follow the project's style guidelines and introduce no
new warnings.
- [x] The changes are fully tested and pass the CI checks.
- [x] I have reviewed my own code changes.

---------

Signed-off-by: burgholzer <burgholzer@me.com>
Co-authored-by: Yannick Stade <100073938+ystade@users.noreply.github.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 usability Anything related to usability

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants