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.

Sign up for GitHub

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

Jump to bottom

[RTD] Fixes documentation #1412

Merged
merged 3 commits into from
Jan 6, 2025
Merged

[RTD] Fixes documentation #1412

merged 3 commits into from
Jan 6, 2025

Conversation

erick-xanadu
Copy link
Contributor

@erick-xanadu erick-xanadu commented Jan 6, 2025 •
edited
Loading

Context:
Before

Description of the Change:

After

Benefits: No floating } and well formatted contents.

Possible Drawbacks: Do we also want to show the libraries we use?

Related GitHub Issues:

@erick-xanadu erick-xanadu changed the title attempt to fix documentation [RTD] Fixes documentation Jan 6, 2025
@erick-xanadu erick-xanadu requested a review from a team January 6, 2025 16:39
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 6, 2025

Hello. You may have forgotten to update the changelog!
Please edit doc/releases/changelog-dev.md on your branch with:

  • A one-to-two sentence description of the change. You may include a small working example for new features.
  • A link back to this PR.
  • Your name (or GitHub username) in the contributors section.

@erick-xanadu erick-xanadu changed the base branch from main to v0.10.0-rc January 6, 2025 16:40
joeycarter
joeycarter reviewed Jan 6, 2025
View reviewed changes
doc/index.rst Outdated Show resolved Hide resolved
joeycarter
joeycarter approved these changes Jan 6, 2025
View reviewed changes
Copy link
Contributor

@joeycarter joeycarter left a comment

Choose a reason for hiding this comment

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

I checked the docs build, looks good now. Thanks! 💯

@erick-xanadu erick-xanadu merged commit 2376fae into v0.10.0-rc Jan 6, 2025
41 checks passed
@erick-xanadu erick-xanadu deleted the eochoa/2025-01-06/read-the-docs branch January 6, 2025 19:46
mehrdad2m pushed a commit that referenced this pull request Jan 6, 2025
@erick-xanadu @mehrdad2m
[RTD] Fixes documentation (#1412)
306d6cb 
**Context:**

![Before](https://github.com/user-attachments/assets/3ec337c1-a898-4abd-a9c6-990d8767679c)

**Description of the Change:**


![After](https://github.com/user-attachments/assets/a07c6a1c-f98d-4b00-b9aa-9afd0a893673)

**Benefits:** No floating `}` and well formatted contents.

**Possible Drawbacks:** Do we also want to show the libraries we use?

**Related GitHub Issues:**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joeycarter joeycarter Awaiting requested review from joeycarter

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@erick-xanadu @joeycarter