Qiskit / qiskit
88%
main: 88%

LAST BUILD BRANCH: gh-readonly-queue/stable/2.2/pr-15163-7304d22fdb19570f553a7b957e5b60dcbba91065
DEFAULT BRANCH: main
Repo Added
Files 867
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH gh-readonly-queue/main/pr-14935-52ebd733a08f60cfde7a26f4fcf6ce27bac5fe5c
branch: gh-readonly-queue/main/pr-14935-52ebd733a08f60cfde7a26f4fcf6ce27bac5fe5c
CHANGE BRANCH
x

coverage: 88.428% (-0.002%) from 88.43%
17154509362

push

github

web-flow 
Rename variable mapper Target enum to SwitchTarget (#14935)

* Rename variable mapper Target enum to ControlFlowTarget

For better or worse we're using cbindgen to generate the C headers and
it does basically a flat search on the entire rust code base for symbol
names when trying to extract details for types exposed across the C FFI.
This means if there are any name overlaps, regardless of whether they're
namespaced or set to private visibility will likely confuse cbindgen and
create an incorrect C header file. The latest instance of this is the
`Target` name. In the C API we export `qiskit_transpiler::target::Target`
as that is the public facing representation of the compilation target to
C, however there was also a `qiskit_circuit::variable_mapper::Target`
enum which is internally used in the circuit crate to represent the
target type of a control flow operation. cbindgen was getting these
confused and exporting the docstring from the enum and not the actual
user facing struct. This commit corrects for this shortcoming by renaming
`qiskit_circuit::variable_mapper::Target` to `ControlFlowTarget` so
there is no longer a name collision and cbindgen will pick the correct
type to use for generating the header.

* Switch from `ControlFlowTarget` to `SwitchTarget`

7 of 12 new or added lines in 1 file covered. (58.33%)

11 existing lines in 4 files now uncovered.

89829 of 101584 relevant lines covered (88.43%)

496594.29 hits per line

Relevant lines Covered
101584 RELEVANT LINES 89829 COVERED LINES
496594.29 HITS PER LINE
Source Files on gh-readonly-queue/main/pr-14935-52ebd733a08f60cfde7a26f4fcf6ce27bac5fe5c

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
17154509362 gh-readonly-queue/main/pr-14935-52ebd733a08f60cfde7a26f4fcf6ce27bac5fe5c Rename variable mapper Target enum to SwitchTarget (#14935) * Rename variable mapper Target enum to ControlFlowTarget For better or worse we're using cbindgen to generate the C headers and it does basically a flat search on the entire rust code ... push web-flow github
88.43
See All Builds (14010)