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

Update testing-debugging.md #648

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Manvi-Agrawal
Copy link
Contributor

@Manvi-Agrawal Manvi-Agrawal commented Apr 19, 2024

Bit of nitpick: The qsharp language header might be a bit confusing to someone new to the Q#. %%qsharp can only be used with jupyter notebook(python kernel with Q#) installed, so it might be helpful to say python, all along the way

Doc link: https://learn.microsoft.com/en-us/azure/quantum/user-guide/testing-debugging

Copy link

Learn Build status updates of commit 2ab4aa4:

✅ Validation status: passed

File Status Preview URL Details
articles/user-guide/testing-debugging.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@Jak-MS
Copy link
Contributor

Jak-MS commented Apr 19, 2024

@Manvi-Agrawal - take this out of draft mode when ready and @Bradben will review.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Jak-MS Jak-MS added the aq-pr-triaged tracking label for the PR review team label Apr 19, 2024
@Bradben
Copy link
Collaborator

Bradben commented Apr 23, 2024

Hi @Manvi-Agrawal - thanks for the feedback. You raise a good point, but the code header needs to reflect the actual code used in the cell, which is Q#, even though it technically is a Python cell. I've opened a work item to clarify the content around the examples to clarify that distinction. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aq-pr-triaged tracking label for the PR review team
Projects
None yet
3 participants