Skip to content

Conversation

@CAM-Gerlach
Copy link
Member

@CAM-Gerlach CAM-Gerlach commented Dec 24, 2025

Pull Request

Pull Request Checklist

  • Read and followed this repo's Contributing Guidelines
  • Based your PR on the latest version of the correct branch
  • Checked your code and writing carefully
  • Described your changes and the motivation for them below
  • Noted what issue(s) this pull request resolves, creating one if needed

Description of Changes

Requires spyder-ide/spyder#25512

  • Thoroughly revise the docstring content for the spyder.api.plugins modules:
    • Ensure all modules and APIs have a proper summary line
    • Document un(der) documented public APIs
    • Greatly clarify and expand existing descriptions
    • Add all param, return and attribute types to both signatures and docstrings
    • Fix and clarify many instances of missing, incorrect or unclear param types in the signature and docstrings related to parameters accepting None and/or having a default value, and clarify in the docstring what that value is and its meaning.
    • Add and fix Sphinx roles and directives throughout
    • Fix all Sphinx broken references in the affected modules

@CAM-Gerlach CAM-Gerlach self-assigned this Dec 24, 2025
@netlify
Copy link

netlify bot commented Dec 24, 2025

Deploy Preview for spyder-api-docs-preview failed.

Name Link
🔨 Latest commit bcfa84e
🔍 Latest deploy log https://app.netlify.com/projects/spyder-api-docs-preview/deploys/6950f29ffb547a0008f59e5a

@CAM-Gerlach CAM-Gerlach force-pushed the revise-plugins-docstrings branch from 1cf8082 to bcfa84e Compare December 28, 2025 09:04
@CAM-Gerlach CAM-Gerlach marked this pull request as ready for review December 28, 2025 09:04
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Thanks @CAM-Gerlach!

@ccordoba12 ccordoba12 merged commit d21a8c5 into spyder-ide:master Dec 28, 2025
4 of 5 checks passed
@CAM-Gerlach CAM-Gerlach deleted the revise-plugins-docstrings branch December 28, 2025 20:40
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