Skip to content

gha: bump doc-strings-review to docs-extensions-and-macros v5.56.0 - #543

Open
JakeSCahill wants to merge 1 commit into
mainfrom
bump-doc-strings-review-v5.56.0
Open

JakeSCahill wants to merge 1 commit into
mainfrom
bump-doc-strings-review-v5.56.0

Conversation

@JakeSCahill

Copy link
Copy Markdown
Contributor

Bumps the doc-strings-review pin from docs-extensions-and-macros v5.55.0 (560c430) to v5.56.0 (90ebfd5) and updates the version note. Follows #539.

What changes for this repo:

  • The connect surface now reads the internal docs.Field* constructors used for shared blocks (http_server cors, tls, logger, batching, api) (docs-extensions-and-macros#417). The version note no longer says those are skipped. Descriptions set as a Description: field in a docs.FieldSpec{} literal are still not read.
  • Lines a human reviewer (not the PR author) wrote on the PR are not reviewed again (public/service: expose collapsed count helpers for plugin developers #396).

No workflow_call input, default, secret, or permission changed, so the shim passes the same values.

Checked:

  • actionlint clean.
  • Lowercased the docs.FieldBool description for cors enabled in a local commit and ran doc-tools lint-strings --surface connect --diff HEAD~1: doc-tools 5.55.0 checks 0 declarations, 5.56.0 checks 1 and reports a starts-lowercase warning.

@claude

claude Bot commented Oct 26, 2026

Copy link
Copy Markdown

Commits
LGTM

Review
Bumps the reusable doc-strings-review workflow to docs-extensions-and-macros v5.56.0. It stays pinned to a commit SHA, with the tag and date annotation the file header asks for. Permissions and secrets have not changed, and the comments match the new behaviour.

LGTM

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