Skip to content

Conversation

@abbycross
Copy link
Collaborator

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@abbycross
Copy link
Collaborator Author

abbycross commented Oct 7, 2025

Unsure why the ./check asked me to add /index to the end of the /docs/guides link in the _toc file. (Errored, saying it could not find the file in the toc)
UPDATE: the check assumes that the index file is always the first in the toc. We want it to be in this case, so moving Quick start to the second entry will fix the error here.

@abbycross
Copy link
Collaborator Author

Ready for review @lerongil @kaelynj !

@abbycross
Copy link
Collaborator Author

@lerongil @kaelynj an issue has been raised about too few qubits for the fake backend in Hello World. We can address that in this PR - do we instruct to install aer from the start, or do we wait till the relevant "alternative: use a simulator" section to suggest it?

@lerongil lerongil moved this to In Progress in Docs Planning Oct 27, 2025
@lerongil lerongil moved this from In Progress to In Review in Docs Planning Oct 27, 2025
@abbycross abbycross requested a review from beckykd October 27, 2025 19:57
@abbycross
Copy link
Collaborator Author

Consensus: @kaelynj will address the hello world fake backend problem separately.

Copy link
Member

@lerongil lerongil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@beckykd beckykd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confetti!

Co-authored-by: Rebecca Dimock <66339736+beckykd@users.noreply.github.com>
Co-authored-by: Rebecca Dimock <66339736+beckykd@users.noreply.github.com>
@abbycross abbycross enabled auto-merge October 27, 2025 20:37
@abbycross abbycross added this pull request to the merge queue Oct 27, 2025
Merged via the queue into main with commit 6b0e089 Oct 27, 2025
4 checks passed
@abbycross abbycross deleted the ajc/quick-start branch October 27, 2025 20:56
@github-project-automation github-project-automation bot moved this from In Review to Done in Docs Planning Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants