search

Qiskit / qiskit-metapackage

Qiskit is an open-source SDK for working with quantum computers at the level of circuits, algorithms, and application modules.
https://qiskit.org
Apache License 2.0
3.03k stars 750 forks source link

Source text issue #1701

Closed owitczak closed 1 year ago

owitczak commented 1 year ago

In Crowdin string English source text https://crowdin.com/translate/qiskit-docs/9536/en-pl#278740 there is a missing ")" before full stop. Here:

Tier 1 platforms are currently: Linux x86_64 (distributions compatible with the manylinux 2014 packaging specification.

https://qiskit.org/documentation/stable/0.34/getting_started.html

I'm not sure if I added this PR correctly, please let me know if I should have submitted this somewhere else.

jakelishman commented 1 year ago

You've made an issue rather than a PR, but this is the correct place. The relevant file that needs fixing is here, if you'd like to fix it? https://github.com/Qiskit/qiskit/blob/master/docs/getting_started.rst

owitczak commented 1 year ago

My apologies! I think I did it correctly now in the link above.

Eric-Arellano commented 1 year ago

Thanks for flagging this @owitczak! To clarify, you would want to make a pull request to this repository with a change to the getting_started.rst file: https://github.com/Qiskit/qiskit/blob/master/docs/getting_started.rst. Updating Crowdin will not fix the original English source.

https://opensource.guide/how-to-contribute/#opening-a-pull-request has a guide for how to open a pull request with open source projects.

atharva-satpute commented 1 year ago

I don't see a PR raised for this issue. Can I take this up?

Eric-Arellano commented 1 year ago

@atharva-satpute, that would be great!