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

[pyright] analyzeUnannotatedFunctions #26392

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

alangenfeld
Copy link
Member

Sibling to https://github.com/dagster-io/internal/pull/12806, enable analyzeUnannotatedFunctions by using a script to insert comments on newly created type errors.

examples is exempted from this as the type suppresion comments negatively impact using terse code snippets in docs etc.

How I Tested These Changes

make pyright

Copy link
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

Copy link
Member

@sbquinlan sbquinlan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, spot checked a few files

@smackesey
Copy link
Collaborator

Where the light shines the roaches coming scurrying out! Hopefully this induces some crowdsourced type-tightening.

@alangenfeld alangenfeld force-pushed the al/12-10-_pyright_analyzeunannotatedfunctions branch 2 times, most recently from 425102d to d85a69e Compare December 10, 2024 23:19
@alangenfeld alangenfeld force-pushed the al/12-10-_pyright_analyzeunannotatedfunctions branch from d85a69e to 1e76ed4 Compare December 11, 2024 15:01
@alangenfeld alangenfeld merged commit b7f826f into master Dec 11, 2024
1 of 2 checks passed
@alangenfeld alangenfeld deleted the al/12-10-_pyright_analyzeunannotatedfunctions branch December 11, 2024 15:19
pskinnerthyme pushed a commit to pskinnerthyme/dagster that referenced this pull request Dec 16, 2024
Sibling to https://github.com/dagster-io/internal/pull/12806, enable
`analyzeUnannotatedFunctions` by using a script to insert comments on
newly created type errors.

`examples` is exempted from this as the type suppresion comments
negatively impact using terse code snippets in docs etc.


## How I Tested These Changes

make pyright
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.

3 participants