Skip to content
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 usage check job on CI (add setuptools dependency) #16423

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

jtpio
Copy link
Member

@jtpio jtpio commented Jun 3, 2024

References

Fixes #16404

Since setuptools is used here:

from setuptools import find_namespace_packages, find_packages

There should likely be a direct dependency on it.

It's possible it was assumed setuptools would be available via one of the dependency, and such dependency does not require it anymore (although not sure which one).

Code changes

Add the setuptools>=40.1.0 dependency.

From : #16404 (comment)

If we end up adding pin we should make it >=40.1.0 as this is when find_namespace_packages was added.

User-facing changes

None

Backwards-incompatible changes

None

Copy link

Thanks for making a pull request to jupyterlab!
To try out this branch on binder, follow this link: Binder

@jtpio jtpio added this to the 4.1.x milestone Jun 3, 2024
@jtpio jtpio marked this pull request as ready for review June 3, 2024 20:54
@jtpio jtpio requested a review from krassowski June 4, 2024 07:55
Copy link
Member

@krassowski krassowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@krassowski krassowski changed the title Troubleshoot usage check on CI Fix usage check job on CI (add setuptools dependency) Jun 4, 2024
@fcollonval fcollonval merged commit 228bd65 into jupyterlab:main Jun 4, 2024
84 checks passed
@fcollonval
Copy link
Member

@meeseeksdev please backport to 4.2.x
@meeseeksdev please backport to 4.1.x

meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Jun 4, 2024
meeseeksmachine pushed a commit to meeseeksmachine/jupyterlab that referenced this pull request Jun 4, 2024
@jtpio jtpio deleted the fix-usage-ci branch June 4, 2024 08:51
jtpio added a commit that referenced this pull request Jun 4, 2024
…dency) (#16428)

Co-authored-by: Jeremy Tuloup <jeremy.tuloup@gmail.com>
jtpio added a commit that referenced this pull request Jun 4, 2024
…dency) (#16429)

Co-authored-by: Jeremy Tuloup <jeremy.tuloup@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Linux Tests / Linux (usage, 3.12) is failing on CI (missing the setup.py file)
3 participants