Fix and refactor imports

mirkolenz commented 1 year ago

Description of change

When installing this package in my library, I noticed that the server of the reflection stubs has an Unknown type. This was caused by a wrongly assigned import. During this fix, I noticed that the current version of this package uses no Literal and this was the only reason the typing-extension package was listed as a dependency. I removed those from the package. Should the need for Literal types arise later on, they can easily be added back. In the meantime, we get a "slimmer" package. In the same stub, I could also remove the sys import since it was not used.

Minimum Reproducible Example

An example is not necessary here as there was no "issue" here. I am not adding/altering any stubs, but only corrected some imports. So no change of functionality here.

Checklist:

[x] I have verified my MRE is sufficient to demonstrate the issue and solution by attempting to execute it myself
[x] I have added tests to typesafety/test_grpc.yml for all APIs added or changed by this PR
[x] I have removed any code generation notices from anything seeded using mypy-protobuf.

mirkolenz commented 1 year ago

Thank you for publishing a release immediately!

shabbyrobe commented 1 year ago

No worries! I had a few headaches after merging related to some other unrelated changes. Please let me know if it breaks anything for you.

mirkolenz commented 1 year ago

I just tested out the new version and so far did not experience any problems.

shabbyrobe / grpc-stubs