1
88%
main: 88%

Ran 22 Aug 2025 12:30PM UTC

Files 848

Run time 28s

Badge

Embed ▾

Committed 22 Aug 2025 11:51AM UTC coverage: 88.428% (-0.002%) from 88.43%

Job # 17154509362.1

Build Type

push

github

Committed by

web-flow

Commit Message

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`

Run Details

89829 of 101584 relevant lines covered (88.43%)

496594.29 hits per line

Qiskit / qiskit / 17154509362 / 1
88%
main: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 17154509362.1

Qiskit / qiskit / 17154509362 / 1 88% main: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 17154509362.1

Qiskit / qiskit / 17154509362 / 1
88%
main: 88%

README BADGES
x