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

Fix examples link in README.md #2242

Merged
merged 4 commits into from
Mar 27, 2024
Merged

Conversation

jordandsullivan
Copy link
Contributor

@jordandsullivan jordandsullivan commented Mar 25, 2024

Under the Calibration section, it says "See our guides and examples for more explanation, techniques, and benchmarks." but the link to examples is currently pointing to our docs homepage (https://mitiq.readthedocs.io/) rather than the examples section (https://mitiq.readthedocs.io/en/stable/examples/examples.html). Simple update.

Second update was to the unitaryHack link, pointing it to the updated 2021 link in the changelog.

Description


License

  • I license this contribution under the terms of the GNU GPL, version 3 and grant Unitary Fund the right to provide additional permissions as described in section 7 of the GNU GPL, version 3.

Before opening the PR, please ensure you have completed the following where appropriate.

It is currently pointing to our docs homepage rather than the examples section.
@jordandsullivan jordandsullivan added documentation Improvements or additions to documentation non-quantum Requires no knowledge of quantum physics to fix / work on. labels Mar 25, 2024
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hello @jordandsullivan, thank you for submitting a PR to Mitiq! We will respond as soon as possible, and if you have any questions in the meantime, you can ask us on the Unitary Fund Discord.

Copy link

codecov bot commented Mar 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.28%. Comparing base (671c8db) to head (e6d5891).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2242   +/-   ##
=======================================
  Coverage   98.28%   98.28%           
=======================================
  Files          90       90           
  Lines        4257     4257           
=======================================
  Hits         4184     4184           
  Misses         73       73           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

The link in like 1305 of the CHANGELOG ([unitaryHACK](https://unitaryfund.github.io/unitaryhack/)!) failed to resolve in the docs build (https://github.com/unitaryfund/mitiq/actions/runs/8425856701/job/23072890372#step:6:1040). Link has now been updated to reflect the new domain for the 2021 unitaryhack website.
* Update unitaryhack link in Changelog to 2021 website
Copy link
Member

@natestemen natestemen left a comment

Choose a reason for hiding this comment

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

Thank you! 🚢

@jordandsullivan jordandsullivan merged commit 6d4ef9b into master Mar 27, 2024
18 checks passed
@jordandsullivan jordandsullivan deleted the jordandsullivan-examples-link branch March 27, 2024 18:11
@nathanshammah
Copy link
Member

Thanks @jordandsullivan!

cosenal pushed a commit that referenced this pull request Jun 26, 2024
* Fix examples link in README.md

It is currently pointing to our docs homepage rather than the examples section.

* Update AUTHORS

* Update link to unitaryHACK in CHANGELOG.md (#2243)

The link in like 1305 of the CHANGELOG ([unitaryHACK](https://unitaryfund.github.io/unitaryhack/)!) failed to resolve in the docs build (https://github.com/unitaryfund/mitiq/actions/runs/8425856701/job/23072890372#step:6:1040). Link has now been updated to reflect the new domain for the 2021 unitaryhack website.

* Update link to 2021 unitaryHACK in CHANGELOG.md (#2246)

* Update unitaryhack link in Changelog to 2021 website
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation non-quantum Requires no knowledge of quantum physics to fix / work on.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants