Skip to content

Improve schema handling in documentation #705

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

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

Conversation

ahal
Copy link
Collaborator

@ahal ahal commented Jun 11, 2025

Alright this one is a bit silly, but it's something that's been bugging me for a really long time. First, this was partly made with AI. I used it to get the basic structure in place for a Sphinx extension that can modify only Schema instances. But then I made heavy manual modifications on top to get it to look the way I wanted and handle all sorts of edge cases.

Second, I don't expect any reviewers to actually dig in and understand how the autoschema.py works. I'm happy to walk through anything that folks want. But maybe a better way to illustrate what it's doing is with some before / after screenshots.

Before:
image

After:
image

@ahal ahal self-assigned this Jun 11, 2025
ahal added 3 commits June 12, 2025 14:00
The `#:` syntax is a special format that tells autodoc to include the
following object. This is better than using `:undoc-members:` which
would otherwise include every global variable in the modules.
@ahal ahal force-pushed the ahal/push-xtusqkostmqv branch from 4d90e79 to 6f7383b Compare June 12, 2025 18:32
@ahal ahal force-pushed the ahal/push-xtusqkostmqv branch from 6f7383b to 4f4e7eb Compare June 12, 2025 19:26
@ahal ahal marked this pull request as ready for review June 12, 2025 20:42
@ahal ahal requested a review from a team as a code owner June 12, 2025 20:42
@ahal ahal requested a review from bhearsum June 12, 2025 20:42
@ahal
Copy link
Collaborator Author

ahal commented Jun 12, 2025

Please don't spend overly long trying to parse autoschema.py, unless you're a masochist.. the bar I'm going for here is "better than before" rather than perfect.

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