Skip to content

fallback types #95

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

Closed
wants to merge 1 commit into from
Closed

fallback types #95

wants to merge 1 commit into from

Conversation

FoamyGuy
Copy link
Contributor

@FoamyGuy FoamyGuy commented Jan 9, 2022

If certain imports used for typing are missing, these custom TypeVars can end up not being defined which causes sphinx to fail to build with them ex: https://github.com/adafruit/Adafruit_CircuitPython_FunHouse/pull/28/checks

This change provides minimal fallbacks for those types so that sphinx build will complete in this case.

Copy link
Member

@anecdata anecdata left a comment

Choose a reason for hiding this comment

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

Regression-tested in a couple of typical GET/POST applications. No change to operation, as expected.

@FoamyGuy
Copy link
Contributor Author

Closing this. As discussed "in the weeds" during the meeting 1/10/21 the plan will be to remove the requirements and define interfaces for the types that are coming from them.

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.

2 participants