-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[tests] utils: refactor type-hint signatures. #12144
[tests] utils: refactor type-hint signatures. #12144
Conversation
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
…efore other imports (PEP8)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks @danieleades! |
tests/utils.py
Outdated
@@ -1,17 +1,19 @@ | |||
from __future__ import annotations | |||
|
|||
import contextlib | |||
__all__ = ["http_server"] | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there should be 1 line after and not two (but I'm not sure with the diff)
…le-level dunder functions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few minor nitpicks, but otherwise all good thanks
…ion. Ref: https://github.com/python/typing/blob/9a39406fa45fbfa76eb5d35afb7863f5f727ad87/docs/source/libraries.rst?plain=1#L108-L119 Co-authored-by: Chris Sewell <chrisj_sewell@hotmail.com>
Thanks @chrisjsewell! |
shall we merge this @jayaddison? |
@chrisjsewell yep, this is ready to merge from my perspective 👍 |
Thank you! |
Feature or Bugfix
Purpose
Detail
http_server
utility method, withtls_enabled
as a boolean flag.Relates