Skip to content

Conversation

@robtaylor
Copy link
Contributor

Summary

  • Add docstrings to SimInterface and Data TypedDict classes to override inherited dict docstrings that cause RST parsing errors
  • Fix IOModelOptions Attributes section formatting with consistent line lengths and proper RST inline literal syntax (double backticks)
  • Fix submodule_metadata docstring to use double backticks for inline literals and proper Args/Yields format
  • Fix Sky130Port docstring with proper RST link syntax and spacing

Context

When building documentation with sphinx-autoapi, TypedDict classes without explicit docstrings inherit Python's built-in dict docstring, which contains indentation that RST interprets as unexpected block quotes, causing 4 ERRORS.

Additionally, several docstrings had issues with:

  • Single backticks (interpreted text) vs double backticks (inline literals)
  • Missing spaces in RST link syntax
  • Definition list formatting in Attributes sections

Test plan

  • Build documentation with pdm run docs in chipflow-docs
  • Verify ERRORS reduced from 4 to 0
  • Verify warnings reduced from 18 to 9

🤖 Generated with Claude Code

- Add docstrings to SimInterface and Data TypedDict classes to override
  inherited dict docstrings that cause RST parsing errors
- Fix IOModelOptions Attributes section formatting with consistent line
  lengths and proper RST inline literal syntax (double backticks)
- Fix submodule_metadata docstring to use double backticks for inline
  literals and proper Args/Yields format
- Fix Sky130Port docstring with proper RST link syntax and spacing

These changes resolve RST errors and warnings generated by sphinx-autoapi
when building documentation.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Dec 17, 2025

PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2025-12-17 00:46 UTC

@github-actions
Copy link

Tests Skipped Failures Errors Time
68 4 💤 0 ❌ 0 🔥 24.181s ⏱️

@robtaylor robtaylor merged commit 5f6ea49 into main Dec 17, 2025
6 checks passed
@robtaylor robtaylor deleted the fix-docstring-formatting branch December 17, 2025 00:45
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.

2 participants