Skip to content

docs: fixes for api docs#35

Merged
obackhouse merged 20 commits into
cm/add_zixy_api_docsfrom
zixy_api_docs_fixes
May 18, 2026
Merged

docs: fixes for api docs#35
obackhouse merged 20 commits into
cm/add_zixy_api_docsfrom
zixy_api_docs_fixes

Conversation

@obackhouse
Copy link
Copy Markdown
Collaborator

No description provided.

@obackhouse obackhouse requested a review from CalMacCQ May 11, 2026 13:21
@obackhouse
Copy link
Copy Markdown
Collaborator Author

also pulls in main

Copy link
Copy Markdown
Collaborator

@CalMacCQ CalMacCQ left a comment

Choose a reason for hiding this comment

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

Looks great, thanks Ollie. I have rebuilt the docs locally on this branch and the number of warnings is dramatically reduced from >3000 to 101. There are still a lot of wanings that look like this.

 WARNING: Explicit markup ends without a blank line; unexpected unindent. [docutils]

I think this could be a whitespace/templating issue. We should consider resolving all of the warnings so we can run a super strict sphinx build in C.I.

Comment thread zixy-py/docs/conf.py Outdated
Comment thread zixy-py/docs/conf.py Outdated
@obackhouse obackhouse requested a review from CalMacCQ May 18, 2026 14:29
@obackhouse
Copy link
Copy Markdown
Collaborator Author

no warnings now, and i'm made warnings error out

@CalMacCQ
Copy link
Copy Markdown
Collaborator

no warnings now, and i'm made warnings error out

You can also accomplish this by passing the -W flag to sphinx-build. In Guppy I have just build and just build-debug where only the latter allows warnings

https://github.com/Quantinuum/guppy-docs/blob/main/justfile#L17

@obackhouse
Copy link
Copy Markdown
Collaborator Author

no warnings now, and i'm made warnings error out

You can also accomplish this by passing the -W flag to sphinx-build. In Guppy I have just build and just build-debug where only the latter allows warnings

https://github.com/Quantinuum/guppy-docs/blob/main/justfile#L17

ah that's probably better place for control, i'll remove and then merge

@obackhouse obackhouse merged commit 4ff2c1a into cm/add_zixy_api_docs May 18, 2026
5 checks passed
@obackhouse obackhouse deleted the zixy_api_docs_fixes branch May 18, 2026 14:47
CalMacCQ added a commit that referenced this pull request May 19, 2026
* add in the sphinx docs build

* update .gitignore to exclude generated files from source control

* move example notebooks into a subdirectory of docs

* add comment to conf.py

* small change to sphinx conf.py

* update conf.py again

* add docs workflow

* update workflow title

* Apply suggestions from code review

Co-authored-by: Oliver Backhouse <olbackhouse@gmail.com>

* ignore checkpoints

* remove 'minimal' cells from notebook examples

* add jupyter as a docs dependency

* mention docs in the README

* change workflow back to run on PRs

* docs: fixes for api docs (#35)

* use strict build in C.I.

* add separate publish workflow that runs on releases

* rm duplciate

* fix missing 'n'

---------

Co-authored-by: Oliver Backhouse <olbackhouse@gmail.com>
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.

3 participants