search

JelleZijlstra / stubdefaulter

Add default values to stubs
Other
10 stars 4 forks source link

Add `--check` argument for CI #96

Closed sobolevn closed 2 months ago

sobolevn commented 2 months ago

We want to integrate stubdefaulter to django-stubs: https://github.com/typeddjango/django-stubs/pull/2327

Later we also want to add stubdefaulter to our CI, so it can check that no missing defaults are added.

--check would help us to fail the CI. We would use both --exit-zero and --check, because there would be errors when importing django modules, we can't fix it from our side. So, we need them both to work correctly. Errors will be ignored with --exit-zero while --check will return 2 if there are any new changes.

github-actions[bot] commented 2 months ago

This change has no effect on how stubdefaulter would alter typeshed's stdlib. 🤖🎉

JelleZijlstra commented 2 months ago

The pyanalyze failures are unrelated, I think I need to cut a new pyanalyze release.