Skip to content

Integrate new & revised docstrings in spyder.api.widgets module#20

Merged
ccordoba12 merged 1 commit intospyder-ide:masterfrom
CAM-Gerlach:revise-widget-docstrings
Feb 5, 2026
Merged

Integrate new & revised docstrings in spyder.api.widgets module#20
ccordoba12 merged 1 commit intospyder-ide:masterfrom
CAM-Gerlach:revise-widget-docstrings

Conversation

@CAM-Gerlach
Copy link
Copy Markdown
Member

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#25598

  • Thoroughly revise the docstring content for the spyder.api.widgets 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
  • Document several other API issues across the touched modules:
    • Add pending deprecation of ToolbarStyle and SpyderMenuProxyStyle and replacement with the corresponding private names, as they were never intended to be used publicly
    • Document not specifying a known TYPE for ToolbarStyle raise a SpyderAPIError rather than silently not working as intended
    • Add admonition in SpyderToolbar to not use it directly in favor of its subclasses ApplicationToolbar and MainWidgetToolbar
    • Document removal of options arg from SpyderActionMixin.update_actions() mixin, which is inconsistent and incompatible with all its actual implementations and already will raise an error at runtime

@CAM-Gerlach CAM-Gerlach self-assigned this Jan 21, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 21, 2026

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

Name Link
🔨 Latest commit 5603428
🔍 Latest deploy log https://app.netlify.com/projects/spyder-api-docs-preview/deploys/6983dc3b54170e000801c69b

@CAM-Gerlach CAM-Gerlach force-pushed the revise-widget-docstrings branch from 9a0b0f7 to 5603428 Compare February 4, 2026 23:54
@CAM-Gerlach CAM-Gerlach marked this pull request as ready for review February 4, 2026 23:54
Copy link
Copy Markdown
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 7120495 into spyder-ide:master Feb 5, 2026
4 of 5 checks passed
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