Skip to content

Conversation

@aborgna-q
Copy link
Collaborator

When the TKET_C_API_PATH env var is set, tket1-passes looks for the shared library in the given path rather than trying to build it by itself.
We use the to be able to cache a single artifact and share it around.

Although the env var is set globally, we weren't calling the install action in a couple jobs.

This should fix the errors seen in #1250

@aborgna-q aborgna-q requested a review from a team as a code owner November 13, 2025 10:34
@aborgna-q aborgna-q requested a review from cqc-alec November 13, 2025 10:34
@aborgna-q aborgna-q added ci CI / devops run-ci-checks Special label to force running all checks in CI labels Nov 13, 2025
@aborgna-q aborgna-q changed the title ci: Add missing tket-c-api installs in CI ci: Add missing tket-c-api installs Nov 13, 2025
@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.11%. Comparing base (49c5996) to head (6fc8cc6).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1254   +/-   ##
=======================================
  Coverage   79.11%   79.11%           
=======================================
  Files         160      160           
  Lines       20412    20413    +1     
  Branches    19480    19481    +1     
=======================================
+ Hits        16149    16150    +1     
  Misses       3280     3280           
  Partials      983      983           
Flag Coverage Δ
python 92.65% <ø> (ø)
qis-compiler 100.00% <ø> (ø)
rust 78.45% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aborgna-q aborgna-q added this pull request to the merge queue Nov 13, 2025
Merged via the queue into main with commit c7bf1c2 Nov 13, 2025
27 checks passed
@aborgna-q aborgna-q deleted the ab/missing-c-api-install branch November 13, 2025 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI / devops run-ci-checks Special label to force running all checks in CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants