-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding from_designator + notes for usage #222
Conversation
In CI this often fails even if it succeeds locally. * Limit the version of triomphe to 0.1.11 because 0.1.12 uses unstable features * Remove some conditional compilation for clippy complaints. Maybe clippy was upgraded independently of rustc?
* Factor lowering a list of qubits into a helper function fixes: Qiskit#163 * Update crates/oq3_semantics/src/syntax_to_semantics.rs remove comments that were moved elsewhere * Update crates/oq3_semantics/src/syntax_to_semantics.rs Move a comment from elsewhere to here, where it makes more sense. * Run cargo fmt --------- Co-authored-by: John Lapeyre <[email protected]>
This PR has a merge conflict. probably because main was updated. I think the updated main branch will fix the CI failure above. |
Just rebased my fork, ideally should have fixed the issues. |
@jlapeyre I see what you mean; I'm reasonably confident I have the changes from main, I think I will make a new PR with a new branch name that doesn't conflict, something like |
Obsolete. Replaced by #224 |
Fixes #170: minor changes because error handling for designators within this file wasn't standardized and should stay separated.