fix: clean-up test dependency in qis compiler release workflow#1191
fix: clean-up test dependency in qis compiler release workflow#1191
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1191 +/- ##
==========================================
- Coverage 78.65% 78.61% -0.05%
==========================================
Files 151 151
Lines 18259 18263 +4
Branches 17169 17169
==========================================
- Hits 14362 14357 -5
- Misses 3017 3026 +9
Partials 880 880
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I am seeing the following error while trying to use this version: ImportError while importing test module '/Users/kartik.singhal/GH/tket2/qis-compiler/python/tests/test_basic_generation.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
../venv-test-arm64/lib/python3.10/site-packages/_pytest/python.py:498: in importtestmodule
mod = import_path(
../venv-test-arm64/lib/python3.10/site-packages/_pytest/pathlib.py:587: in import_path
importlib.import_module(module_name)
/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/importlib/__init__.py:126: in import_module
return _bootstrap._gcd_import(name[level:], package, level)
<frozen importlib._bootstrap>:1050: in _gcd_import
???
<frozen importlib._bootstrap>:1027: in _find_and_load
???
<frozen importlib._bootstrap>:1006: in _find_and_load_unlocked
???
<frozen importlib._bootstrap>:688: in _load_unlocked
???
../venv-test-arm64/lib/python3.10/site-packages/_pytest/assertion/rewrite.py:186: in exec_module
exec(co, module.__dict__)
/Users/kartik.singhal/GH/tket2/qis-compiler/python/tests/test_basic_generation.py:2: in <module>
from guppylang import guppy
../venv-test-arm64/lib/python3.10/site-packages/guppylang/__init__.py:5: in <module>
from guppylang.std import builtins, debug, quantum
../venv-test-arm64/lib/python3.10/site-packages/guppylang/std/builtins.py:3: in <module>
from guppylang.std.array import ArrayIter, FrozenarrayIter, array, frozenarray
../venv-test-arm64/lib/python3.10/site-packages/guppylang/std/array.py:18: in <module>
from guppylang_internals.std._internal.compiler.array import (
E ImportError: cannot import name 'ArrayDiscardAllUsedCompiler' from 'guppylang_internals.std._internal.compiler.array' (/private/var/folders/02/hfzwwrm95v30shqp0f12hsf00000gq/T/cibw-run-mr33wn55/cp310-macosx_arm64/venv-test-arm64/lib/python3.10/site-packages/guppylang_internals/std/_internal/compiler/array.py)
Worth ensuring tests pass locally with |
Release-as: 0.2.8
yes, ideally I think we should run in CI for PRs that change edit: Now included in this PR |
726bc33 to
5168e9c
Compare
10bccb4 to
e070f93
Compare
e070f93 to
ccde323
Compare
🤖 I have created a release *beep* *boop* --- ## [0.2.8](qis-compiler-v0.2.7...qis-compiler-v0.2.8) (2025-10-21) ### Bug Fixes * clean-up test dependency in qis compiler release workflow ([#1191](#1191)) ([61bc472](61bc472)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: Seyon Sivarajah <seyon.sivarajah@quantinuum.com>
Release-As: 0.2.8