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

Remove dependency on typeguard #411

Merged
merged 4 commits into from
Nov 10, 2023

Conversation

karthiknadig
Copy link
Contributor

Description (e.g. "Related to ...", etc.)

Remove dependency on typeguard.
fixes #409

Code review checklist (for code reviewer to complete)

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Commit messages are meaningful (see this for details)
  • Tests have been included and/or updated, as appropriate
  • Docstrings have been included and/or updated, as appropriate
  • Standalone docs have been updated accordingly

@karthiknadig karthiknadig marked this pull request as ready for review November 10, 2023 19:58
@tombh
Copy link
Collaborator

tombh commented Nov 10, 2023

Great, thanks. Just request a review from me or Alex when this is ready.

@karthiknadig
Copy link
Contributor Author

@tombh I don't have permissions to add reviewers. This is ready now for review.

@tombh tombh requested review from tombh and alcarney November 10, 2023 21:11
@tombh
Copy link
Collaborator

tombh commented Nov 10, 2023

Oh, sorry, I didn't realise. It looks good, I've approved it. I'll just wait a bit in case Alex gets a chance to look too.

@tombh tombh merged commit 7e9c272 into openlawlibrary:main Nov 10, 2023
17 checks passed
@tombh
Copy link
Collaborator

tombh commented Nov 10, 2023

Would you like a Pygls release? Or wait a bit for some bleeding edge testers?

@karthiknadig karthiknadig deleted the remove-typeguard branch November 10, 2023 23:59
@karthiknadig
Copy link
Contributor Author

I will do more tests with a few servers and let you know.

@karthiknadig
Copy link
Contributor Author

karthiknadig commented Nov 16, 2023

I tested this on a few different servers. It seems to be working as expected. This can be published.

tombh added a commit that referenced this pull request Nov 16, 2023
* Remove dependency on `typeguard` by @karthiknadig in #411
* Simplify vscode-playground setup and fix Python discovery by @alcarney in #374
@tombh tombh mentioned this pull request Nov 16, 2023
tombh added a commit that referenced this pull request Nov 16, 2023
* Remove dependency on `typeguard` by @karthiknadig in #411
* Simplify vscode-playground setup and fix Python discovery by @alcarney in #374
@tombh
Copy link
Collaborator

tombh commented Nov 16, 2023

Great, thank you. Here's the PR for the release #412

tombh added a commit that referenced this pull request Nov 18, 2023
* Remove dependency on `typeguard` by @karthiknadig in #411
* Simplify vscode-playground setup and fix Python discovery by @alcarney in #374
tombh added a commit that referenced this pull request Nov 18, 2023
* Remove dependency on `typeguard` by @karthiknadig in #411
* Simplify vscode-playground setup and fix Python discovery by @alcarney in #374
tombh added a commit that referenced this pull request Nov 18, 2023
* Remove dependency on `typeguard` by @karthiknadig in #411
* Simplify vscode-playground setup and fix Python discovery by @alcarney in #374
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.

Is it possible to drop typeguard?
3 participants