Skip to content
This repository was archived by the owner on Aug 21, 2023. It is now read-only.

Small fixes of 1_getting_started_with_qiskit.ipynb#1383

Merged
mergify[bot] merged 2 commits intoQiskit:masterfrom
filippotramonto:master
Jan 12, 2023
Merged

Small fixes of 1_getting_started_with_qiskit.ipynb#1383
mergify[bot] merged 2 commits intoQiskit:masterfrom
filippotramonto:master

Conversation

@filippotramonto
Copy link
Copy Markdown
Contributor

Summary

Small fixes of the tutorial 1_getting_started_with_qiskit:

  • "execute function" --> "run method" in the text (in the code cells was already correct)
  • "addition operator" --> "compose method" in a comment of a code cell
  • there was some backtick too much, so the fix is: "" --> "`" (for example, job`` --> job)

Details and comments

see above

Small fixes of the tutorial 1_getting_started_with_qiskit:
- "`execute` function" --> "`run` method" in the text (in the code cells was already correct)
- "addition operator" --> "compose method" in a comment of a code cell
- there was some backtick too much, so the fix is: "``" --> "`" (for example, ``job`` --> `job`)
@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

Copy link
Copy Markdown

@arne-kapell arne-kapell left a comment

Choose a reason for hiding this comment

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

only the described small fixes, nothing is changed code-wise

(I noticed the mistakes too, so I went looking for an already opened PR)

Copy link
Copy Markdown
Contributor

@HuangJunye HuangJunye left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the fix!

@HuangJunye
Copy link
Copy Markdown
Contributor

@arne-kapell Thanks for the review as well. Your efforts are appreciated!

@mergify mergify Bot merged commit fa6de99 into Qiskit:master Jan 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants