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 and make minor additions to docs/ #7175

Open
wants to merge 25 commits into
base: main
Choose a base branch
from

Conversation

mhucka
Copy link
Contributor

@mhucka mhucka commented Mar 22, 2025

Changes:

  • Removed mention of Rigetti from indexes and other places, but left docs/hardware/rigetti. The intention is to stop advertising it because it's being deprecated, but leave the docs so that people who still use it can find some documentation.

  • Added a statement at the top of all the files in docs/hardware/rigetti stating it's deprecated in this release.

  • In the versioning process document, clarified the backwards compatibility policy w.r.t. hardware vendor interface modules.

  • Added mention of new Tesseract Decoder project in a couple of relevant places.

  • Made updates to the triage process document.

  • Made updates to the tool ecosystem page.

  • Made updates to the support page with more info.

  • Fixed tiny typos, capitalization inconsistencies, missing hyphens, tiny formatting niggles like putting a few more things in code face.

mhucka added 8 commits March 21, 2025 16:19
With the deprecation of the cirq-rigetti module in Cirq 1.5, having
sections about it in the documentation seems like it would be
confusion to readers.
It seems like the Docker container may not be used solely for the
Rigetti module, so I left the other text about Docker and only removed
the statement involving Rigetti.
This entire section still needs a good re-think and more revisions.
For now, I just removed mention of the labels that have been changed.
It's true that the section on what's covered mentioned that vendor
modules are not, but it seemed strange that it was not mentioned in
the section that is specifically about what's no covered.
Mostly fix capitalization and occasional missing hyphens.
Copy link

codecov bot commented Mar 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.13%. Comparing base (71c3cf1) to head (400195c).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7175   +/-   ##
=======================================
  Coverage   98.13%   98.13%           
=======================================
  Files        1095     1095           
  Lines       95649    95649           
=======================================
  Hits        93867    93867           
  Misses       1782     1782           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

mhucka added 10 commits March 22, 2025 16:03
Putting the bullet item about hardware modules seems higher in the
list seems like it's better, so that the flow of concepts goes
somewhare more general to somewhat more specific.
Changes:

* There were 3 H1 headings, and at the same time, the first heading
  did not match what is in the TOC. I changed this to have one H1
  heading that matches the TOC text, made the remaining headings H2,
  and added a lead-in paragaraph

* Rewrote the section about using the issue tracker in attempt to be
  more specific and clear.

* Added a section about using the Q.C. Stack Exchange.

* Added a section about using email to contact us.
A trivial change to match the way other headings were written.
Removed more no-longer-existing issue labels, and rewrote section on
stale issue/PR heading to be more succinct.
Capitalization and placement of colon characters.
@mhucka mhucka marked this pull request as ready for review March 22, 2025 23:48
@mhucka mhucka requested a review from pavoljuhas March 22, 2025 23:49
@mhucka mhucka changed the title Do minor updates and additions to docs/ Update and make minor additions to docs/ Mar 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant