Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge typeshed style guide w/ official typing docs? #1880

Open
yangdanny97 opened this issue Nov 6, 2024 · 5 comments
Open

Merge typeshed style guide w/ official typing docs? #1880

yangdanny97 opened this issue Nov 6, 2024 · 5 comments
Labels
topic: documentation Documentation-related issues and PRs

Comments

@yangdanny97
Copy link
Contributor

I see that contributing.md has a style guide for stubs, which somewhat duplicates the documentation on Writing and Maintaining Stubs on the official typing docs.

Should we reduce duplication by linking to the latter from the former? There might be a few extra typeshed-specific things, but for general style stuff I don't see why we need to write it all out again here.

@JelleZijlstra
Copy link
Member

Yes, we should point people to the typing docs, and only add typeshed-specific guidance where it's different from the general guide.

@srittau
Copy link
Collaborator

srittau commented Nov 6, 2024

In fact, the current style guide from the typing docs was originally copied from typeshed. But due to the limbo that the stubs PEP – which ultimately became part of the docs – was in for a long time, the removal of the advice from the typeshed docs never happened. Most advice from CONTRIBUTING probably needs to be merged into the typing docs and then CONTRIBUTING needs slimming down.

@srittau srittau transferred this issue from python/typeshed Nov 6, 2024
@srittau srittau changed the title Merge stub style guide w/ official typing docs? Merge typeshed style guide w/ official typing docs? Nov 6, 2024
@srittau srittau added the topic: documentation Documentation-related issues and PRs label Nov 6, 2024
@yangdanny97
Copy link
Contributor Author

I can take a stab at merging these docs.

The instructions on how to handle disagreement between docs and implementation seems to differ between the two docs.

CONTRIBUTING.md says to follow the implementation over the docs, while the typing guide just says to use your best judgement about intent.

https://github.com/python/typeshed/blob/main/CONTRIBUTING.md#what-to-do-when-a-projects-documentation-and-implementation-disagree

https://typing.readthedocs.io/en/latest/guides/writing_stubs.html#documentation-or-implementation

@yangdanny97
Copy link
Contributor Author

For the other stuff, made a PR here: #1882

There are a few typeshed-specific sections that probably need to remain:

  • Incomplete annotations
  • Any vs Incomplete
  • The Any trick

@yangdanny97
Copy link
Contributor Author

The guide to remove docstrings for conciseness seems to conflict with Pyright's advice here: https://microsoft.github.io/pyright/#/typed-libraries?id=inlined-type-annotations-and-type-stubs

Not sure how/if we should align that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: documentation Documentation-related issues and PRs
Projects
None yet
Development

No branches or pull requests

3 participants