-
Notifications
You must be signed in to change notification settings - Fork 906
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
Fix Failing CI tests caused by new setuptools release #2269
Conversation
Signed-off-by: SajidAlamQB <90610031+SajidAlamQB@users.noreply.github.com>
Signed-off-by: SajidAlamQB <90610031+SajidAlamQB@users.noreply.github.com>
Signed-off-by: SajidAlamQB <90610031+SajidAlamQB@users.noreply.github.com>
Is there a link to a failing CI job that shows the failure? Or, alternatively, what's exactly the error message? |
It's failing for all PRs here's an example. You can recreate it manually by running the tests locally as well. The actual test that's failing is |
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 is a sufficient solution for now, but we should add an issue and prioritise fixing this properly. Ideally we don't have to bound things like pip
or setuptools
.
Signed-off-by: SajidAlamQB <90610031+SajidAlamQB@users.noreply.github.com>
Signed-off-by: SajidAlamQB <90610031+SajidAlamQB@users.noreply.github.com>
Signed-off-by: SajidAlamQB <90610031+SajidAlamQB@users.noreply.github.com>
Signed-off-by: SajidAlamQB 90610031+SajidAlamQB@users.noreply.github.com
Description
CI is failing on
test_complex_requirements
intest_micropkg_requirments.py
due to the latest release ofsetuptools
67.0.0 which has made some changes topkg_resources
. Adding an upper bound tosetuptools
pin resolves it for now.Development notes
Pinned
setuptools>=65.5.1, <67.0.0
Fixed PyLint errors example, instead of
dict()
use literals{...}
Pinned
jupyterlab~=3.0, <3.6.0
as3.6.0
requiresjupyterlab_server~=2.19
Checklist
RELEASE.md
file