Skip to content

🔧 Update artifact download pattern in CI workflow#870

Merged
burgholzer merged 1 commit into
mainfrom
fix_cd
Mar 23, 2026
Merged

🔧 Update artifact download pattern in CI workflow#870
burgholzer merged 1 commit into
mainfrom
fix_cd

Conversation

@simon1hofmann
Copy link
Copy Markdown
Collaborator

Description

Creating archives when uploading artifacts was selectively disabled (munich-quantum-toolkit/workflows#332) leading to artifact name mismatches.

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.

If PR contains AI-assisted content:

  • I have disclosed the use of AI tools in the PR description as per our AI Usage Guidelines.
  • AI-assisted commits include an Assisted-by: [Model Name] via [Tool Name] footer.
  • I confirm that I have personally reviewed and understood all AI-generated content, and accept full responsibility for it.

Copy link
Copy Markdown
Member

@burgholzer burgholzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix. Let's try again with the release.

@burgholzer burgholzer merged commit a6ef088 into main Mar 23, 2026
13 of 14 checks passed
@burgholzer burgholzer deleted the fix_cd branch March 23, 2026 15:32
@github-project-automation github-project-automation Bot moved this from In Progress to Done in MQT Applications Mar 23, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 23, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Updated deployment workflow artifact handling to ensure correct build artifacts are processed during PyPI publishing.

Walkthrough

The PyPI deployment workflow's artifact download pattern was updated from cibw-* to mqt_bench-*, changing which build artifacts are collected from the dist/ directory before attestation generation and PyPI publication.

Changes

Cohort / File(s) Summary
CI/CD Workflow Configuration
.github/workflows/cd.yml
Updated artifact download pattern in PyPI deploy step from cibw-* to mqt_bench-* to target project-specific build artifacts.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hop, hop! The artifacts now know their true name,
From cibw to mqt_bench—a game-changing aim,
With patterns updated, they swiftly deploy,
To PyPI's embrace, oh what joy! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description provides context about the artifact name mismatch issue caused by disabling archive creation, with a reference to the related change. However, most checklist items are unchecked, indicating incomplete preparation. Complete the checklist items by checking off applicable boxes and clarifying whether tests, documentation, or changelog updates are needed for this workflow change.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: updating the artifact download pattern in the CI workflow. It accurately represents the core modification in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix_cd

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@burgholzer burgholzer added continuous integration Anything related to the CI setup fix Fixes something that isn't working labels Mar 23, 2026
denialhaag added a commit that referenced this pull request Apr 8, 2026
@denialhaag denialhaag changed the title 🔧 Update artifact download pattern in CI workflow 🔧 Update artifact download pattern in CI workflow Apr 8, 2026
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 fix Fixes something that isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants