-
Notifications
You must be signed in to change notification settings - Fork 92
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
attrs
version
#102
Comments
Well, there were a few breaking changes that caused us to limit the version (IIRC it was related to the |
Hello @jkhenning , We would also really appreciate freeing up the upper version limit for AFAIK it should not break any other functionality except the List of breaking changes can be seen here - https://www.attrs.org/en/stable/changelog.html#id36 And if I am not mistaken, there's no other changes to the So what do you think? Could we create a PR to release the upper limit? |
Hi @Honzys, Sure, we would welcome a PR 🙂 👍 |
Hi there,
Is there any reason for this upperbound?
clearml-agent/requirements.txt
Line 1 in e3e6a1d
Looking at the release log I don't think
clearml-agent
will be affected by any of the breaking changes, but I'm not an expert!If it can be upgraded to
attrs>=18.0,<21.5.0
I'll make a PR.The text was updated successfully, but these errors were encountered: