Skip to content

Minor style changes to qvm builder code notebook#5808

Merged
CirqBot merged 1 commit intoquantumlib:masterfrom
augustehirth:hide
Jul 22, 2022
Merged

Minor style changes to qvm builder code notebook#5808
CirqBot merged 1 commit intoquantumlib:masterfrom
augustehirth:hide

Conversation

@augustehirth
Copy link
Copy Markdown
Collaborator

Hide some of the code behind a button in the builder notebook, use the same example shown in QSS, add an explanatory comment in notebook_test

@augustehirth augustehirth requested review from a team, cduck and vtomole as code owners July 22, 2022 18:02
@augustehirth augustehirth requested a review from pavoljuhas July 22, 2022 18:02
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@CirqBot CirqBot added the Size: XS <10 lines changed label Jul 22, 2022
@MichaelBroughton MichaelBroughton added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Jul 22, 2022
@MichaelBroughton MichaelBroughton self-assigned this Jul 22, 2022
@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Jul 22, 2022
@CirqBot CirqBot merged commit 3d61112 into quantumlib:master Jul 22, 2022
@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 22, 2022
@augustehirth augustehirth deleted the hide branch July 22, 2022 19:15
rht pushed a commit to rht/Cirq that referenced this pull request May 1, 2023
Hide some of the code behind a button in the builder notebook, use the same example shown in QSS, add an explanatory comment in notebook_test
@augustehirth augustehirth restored the hide branch August 25, 2023 17:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: XS <10 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants