-
Notifications
You must be signed in to change notification settings - Fork 64
minimum python3.10 and pytest8.0 #192
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
base: main
Are you sure you want to change the base?
minimum python3.10 and pytest8.0 #192
Conversation
flub
left a comment
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.
Thanks for the follow-up! LGTM but I think one spelling check was resolved the wrong way.
pytest_timeout.py
Outdated
| """Stop the timeouts when we entered pdb. | ||
| This stops timeouts from triggering when pytest's builting pdb | ||
| This stops timeouts from triggering when pytest's building pdb |
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.
| This stops timeouts from triggering when pytest's building pdb | |
| This stops timeouts from triggering when pytest's builtin pdb |
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.
fixed this, thanks
And change black 25.9.0 to 25.11.0
…nto minimum_python310_and_pytest8 # Conflicts: # .pre-commit-config.yaml # pytest_timeout.py
…nto minimum_python310_and_pytest8
|
Hum, could you resolve the conflicts that github claims there are? |

#191 (comment)