Skip to content
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

Remove key protocols frozenset conversion #5660

Merged
merged 4 commits into from
Jul 7, 2022

Conversation

daxfohl
Copy link
Collaborator

@daxfohl daxfohl commented Jul 1, 2022

Removes the frozenset conversion since the protocol spec requires FrozenSet now. Also removed a deprecated parameter in act_on since it's the only other 0.16 thing in protocols. @95-martin-orion

@daxfohl daxfohl requested review from a team, vtomole and cduck as code owners July 1, 2022 00:37
@daxfohl daxfohl requested a review from 95-martin-orion July 1, 2022 00:37
@CirqBot CirqBot added the size: M 50< lines changed <250 label Jul 1, 2022
@95-martin-orion 95-martin-orion added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Jul 7, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Jul 7, 2022
@CirqBot
Copy link
Collaborator

CirqBot commented Jul 7, 2022

Automerge cancelled: A required status check is not present.

Missing statuses: ['Misc check', 'Notebook formatting', 'Pytest Ubuntu (3.7)', 'Pytest Ubuntu (3.8)', 'Pytest Ubuntu (3.9)', 'Pytest Windows (3.7)', 'Pytest Windows (3.8)', 'Pytest Windows (3.9)', 'Type check', 'Typescript lint check', 'Typescript tests', 'Typescript tests coverage']

@CirqBot CirqBot removed automerge Tells CirqBot to sync and merge this PR. (If it's running.) front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. labels Jul 7, 2022
@daxfohl daxfohl requested a review from 95-martin-orion July 7, 2022 16:54
@95-martin-orion 95-martin-orion merged commit 2657c53 into quantumlib:master Jul 7, 2022
@daxfohl daxfohl deleted the dep-frozenset branch August 12, 2022 05:15
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
Co-authored-by: Cirq Bot <craiggidney+github+cirqbot@google.com>
harry-phasecraft pushed a commit to PhaseCraft/Cirq that referenced this pull request Oct 31, 2024
Co-authored-by: Cirq Bot <craiggidney+github+cirqbot@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: M 50< lines changed <250
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants