Skip to content

cleanups + bugfixes#996

Merged
RonnyPfannschmidt merged 18 commits intopypa:mainfrom
RonnyPfannschmidt:ronny/cleanups
Feb 15, 2024
Merged

cleanups + bugfixes#996
RonnyPfannschmidt merged 18 commits intopypa:mainfrom
RonnyPfannschmidt:ronny/cleanups

Conversation

@RonnyPfannschmidt
Copy link
Contributor

@RonnyPfannschmidt RonnyPfannschmidt commented Jan 5, 2024

@eachimei
Copy link

eachimei commented Jan 5, 2024

@RonnyPfannschmidt please kindly note that in git.py in run_git there's currently a default value to timeout: int = 20. As a result, the changes of BROKEN_TIMEOUT (env var override) won't take effect.
Also, in fetch_shallow() a timeout value of 240 is passed into run_git call so if the user overrides the env-var it won't impact this subprocess call.

@RonnyPfannschmidt
Copy link
Contributor Author

thanks for the note, - i completely missed that detail - i'll integrate the none pass-over there as well

@RonnyPfannschmidt
Copy link
Contributor Author

i'll leave the timeout i fetch shallow for now as its intentional as is and the delay should work as intended for now

its not used by default and i'll start to recommend people prepare unshallow in ci instead of adding the setting to setuptools_scm

@eachimei
Copy link

eachimei commented Jan 8, 2024

alright, sounds good 👍

@RonnyPfannschmidt RonnyPfannschmidt force-pushed the ronny/cleanups branch 2 times, most recently from 1ae2ec1 to 9af3ba4 Compare January 8, 2024 13:28
@RonnyPfannschmidt RonnyPfannschmidt merged commit d9b3b29 into pypa:main Feb 15, 2024
@RonnyPfannschmidt RonnyPfannschmidt deleted the ronny/cleanups branch February 15, 2024 23:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants