Skip to content

Conversation

@CAM-Gerlach
Copy link
Member

@CAM-Gerlach CAM-Gerlach commented Dec 27, 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

Description of Changes

Requires spyder-ide/spyder#25515

  • Thoroughly revise the docstring content for the spyder.api.shellconnect 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 27, 2025
@netlify
Copy link

netlify bot commented Dec 27, 2025

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

Name Link
🔨 Latest commit a334b1b
🔍 Latest deploy log https://app.netlify.com/projects/spyder-api-docs-preview/deploys/6955ba53b5b6be00082aa660

@CAM-Gerlach CAM-Gerlach force-pushed the revise-shellconnect-docstrings branch from c130c40 to a334b1b Compare January 1, 2026 00:05
@CAM-Gerlach CAM-Gerlach marked this pull request as ready for review January 1, 2026 00:05
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 ecd9208 into spyder-ide:master Jan 1, 2026
4 of 5 checks passed
CAM-Gerlach added a commit to CAM-Gerlach/spyder-api-docs that referenced this pull request Jan 1, 2026
CAM-Gerlach added a commit to CAM-Gerlach/spyder-api-docs that referenced this pull request Jan 1, 2026
CAM-Gerlach added a commit to CAM-Gerlach/spyder-api-docs that referenced this pull request Jan 2, 2026
CAM-Gerlach added a commit to CAM-Gerlach/spyder-api-docs that referenced this pull request Jan 2, 2026
CAM-Gerlach added a commit to CAM-Gerlach/spyder-api-docs that referenced this pull request Jan 2, 2026
@CAM-Gerlach CAM-Gerlach deleted the revise-shellconnect-docstrings branch January 6, 2026 01:24
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