Skip to content

List every parameter in every port - #159

Closed
tony wants to merge 1 commit into
mainfrom
document-all-parameters
Closed

tony wants to merge 1 commit into
mainfrom
document-all-parameters

Conversation

@tony

@tony tony commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

Python, Swift, Ruby and Lua listed only the parameters a doc comment described. The Ruby Server.start page therefore had no Parameters section for executable:, config:, timeout: and cancel:, while C# listed options (ServerConnectionOptions?) whether or not it was documented.

Every port now lists each declared parameter with its type, and every parameter name in the signature links to its entry.

  • Python lists the implementation's parameters rather than every @overload stub's, which repeat them with narrower types.
  • An undocumented leading self, or cls in Python, is the receiver the call never passes, so it is not listed or linked. Lua's self is handled the same way.
  • C#'s params arrays still list only the declared name.

Tests: the tests that pinned documented-only lists for Ruby, Lua, Swift and Python now assert the new rule, and a new test covers the receiver rule. The full site suite passes (4,600 tests), and lint passes.

why: Python, Swift, Ruby and Lua listed only the parameters a doc comment
described, so a Ruby Server.start page showed no Parameters section for
executable:, config:, timeout: and cancel:, while C# listed its
parameters with their types whether documented or not.

what:
- List each declared parameter with its type in every port
- Link every parameter name in the signature to its entry
- For Python, list the implementation's parameters, not each @overload
  stub's, and leave out an undocumented self or cls receiver; leave out
  Lua's self receiver too
- Update the tests that pinned documented-only lists
@tony

tony commented Oct 18, 2026

Copy link
Copy Markdown
Contributor Author

Folded into #157, which now carries every fix from the 2026-10-18 review on one branch so they share one CI run, preview and deploy. This commit is on that branch unchanged. Discussion and evidence stay here.

@tony tony closed this Oct 18, 2026
@tony
tony deployed to docs-preview-cleanup October 18, 2026 11:27 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
docs-preview-cleanup — c3ac34e6 Deployed Oct 18, 2026 by tony via cleanup #143
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