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

Revert publishing types #521

Merged
merged 2 commits into from
Aug 31, 2023
Merged

Revert publishing types #521

merged 2 commits into from
Aug 31, 2023

Conversation

adamjmcgrath
Copy link
Contributor

@adamjmcgrath adamjmcgrath commented Aug 30, 2023

Changes

This package started publishing types in 4.4, but the way we add async classes and methods makes type checking unfeasible.

Revert publishing types externally until a better solution is available (this will probably be by generating the SDK).

References

fixes #514
See https://mypy.readthedocs.io/en/stable/installed_packages.html#creating-pep-561-compatible-packages

@adamjmcgrath adamjmcgrath requested a review from a team as a code owner August 30, 2023 12:26
@adamjmcgrath adamjmcgrath temporarily deployed to internal August 30, 2023 12:26 — with GitHub Actions Inactive
@adamjmcgrath adamjmcgrath temporarily deployed to internal August 30, 2023 12:26 — with GitHub Actions Inactive
@adamjmcgrath adamjmcgrath temporarily deployed to internal August 30, 2023 12:26 — with GitHub Actions Inactive
@@ -6,7 +6,7 @@ ignore_errors = True

[mypy-auth0.management.*]
ignore_errors = False
disable_error_code=var-annotated, attr-defined
disable_error_code=var-annotated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

revert #515 since this wasn't a fix for #514

@adamjmcgrath adamjmcgrath temporarily deployed to internal August 30, 2023 12:29 — with GitHub Actions Inactive
@adamjmcgrath adamjmcgrath temporarily deployed to internal August 30, 2023 12:32 — with GitHub Actions Inactive
@adamjmcgrath adamjmcgrath temporarily deployed to internal August 30, 2023 12:32 — with GitHub Actions Inactive
@adamjmcgrath adamjmcgrath temporarily deployed to internal August 30, 2023 12:32 — with GitHub Actions Inactive
@adamjmcgrath adamjmcgrath added this pull request to the merge queue Aug 31, 2023
Merged via the queue into master with commit 8c75c6c Aug 31, 2023
15 checks passed
@adamjmcgrath adamjmcgrath deleted the revert-types branch August 31, 2023 08:24
@adamjmcgrath adamjmcgrath mentioned this pull request Aug 31, 2023
github-merge-queue bot pushed a commit that referenced this pull request Sep 5, 2023
**Fixed**
- Fix python dependency version
[\#522](#522)
([adamjmcgrath](https://github.com/adamjmcgrath))
- Revert publishing types
[\#521](#521)
([adamjmcgrath](https://github.com/adamjmcgrath))
@adamjmcgrath adamjmcgrath mentioned this pull request Oct 31, 2023
5 tasks
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.

Type checks started failing on our codebase with v4.4
2 participants