Qiskit
/
qiskit
/
30016545767
88%
main: 88%
main: 88%
|
Ran
|
Jobs
1
|
Files
950
|
Run time
2min
|
Badge
Embed ▾
README BADGES
|
coverage: 87.628% (+0.02%) from 87.613%
push
github
web-flowSet the C language spec used during C tests (#16562) (#16643) Our `CMakeLists.txt` file was previously leaving this undefined, and so using the defaults of whatever build tool came along. This could mean that newer toolchains would build in different modes, and thus see different behaviours. Here, we set the test defaults to match exactly Qiskit's minimum C compiler requirements without extensions. This recently occurred for Qiskit v2.5, where the bump to `cbindgen==0.29.3`[^1] changed the behaviour of fixed-width `enum` exports to use the new C23 / C++-like `enum X : <ty>` syntax, gated on the reported language-spec[^2]. Recent `gcc` (15+) default to `-std=gnu23`, so packagers for recent Linux distros saw new build failures in the C test suite that were uncaught by CI[^3]. A follow-up commit can modify our CI to explicitly test a range of C language versions. [^1]: 3e8d124e0b: Bump cbindgen from 0.29.2 to 0.29.3 (#16300) [^2]: mozilla/cbindgen@b16c1d2025: Use C++ fixed-type enumeration syntax under C23 (or higher) as well [^3]: https://github.com/Qiskit/qiskit/issues/16555 (cherry picked from commit 9eacb5427) Co-authored-by: Jake Lishman <jake.lishman@ibm.com>
112833 of 128764 relevant lines covered (87.63%)
997254.18 hits per line
Coverage Regressions
| Lines | Coverage | ∆ | File |
|---|---|---|---|
| 3 |
92.8
|
1.03% | crates/qasm2/src/lex.rs |
Jobs
| ID | Job ID | Ran | Files | Coverage | |
|---|---|---|---|---|---|
| 1 | 30016545767.1 | 950 |
87.63
|
GitHub Action Run |
