-
Notifications
You must be signed in to change notification settings - Fork 32
Bug 1596439 - Update dependencies for scriptworker 29.0.1 #80
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
Conversation
Hm, I wonder if the treescript requirement changes means we won’t install scriptworker at all? In which case we may need to add scriptworker back in, even if we don’t import it. |
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.
Yeah, this will break docker images and treescript won't work. Can you explicitly add scriptworker to treescript's base requirements?
52c7180
to
5e7e911
Compare
Great catch @escapewindow! I added |
I didn't think about it til I saw your diff :) Sorry I removed it. |
scriptworker==29.0.0 \ | ||
--hash=sha256:32d06bedaa7f8134cdb6319ff4c1d044c03472ba3145b971989a52527858b274 \ | ||
--hash=sha256:91f999f3eb60f7fd43b5e8df3f260ed79d277a930823a33de3618692d90ba89b | ||
scriptworker==29.0.1 \ |
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.
Hm. I need 29.0.2 on treescript. (I just released that for https://bugzilla.mozilla.org/show_bug.cgi?id=1597671 ).
We can push your change, and then I can push my change to production-treescript, or we can push once with 29.0.2. I'm running pin.sh right now.
2 notable things in addition to the regular bump:
importlib-metadata
is not a dependency anymore becausejsonschema
removed it for python 3.8+ Use importlib from stdlib (if possible) python-jsonschema/jsonschema#627. Thus,zipp
andmore-itertools
are gone too.