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.

Sign up for GitHub

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

Jump to bottom

Fix CI failure #820

Closed
wants to merge 2 commits into from
Closed

Fix CI failure #820

wants to merge 2 commits into from

Conversation

Jasha10
Copy link
Collaborator

@Jasha10 Jasha10 commented Nov 15, 2021 •
edited

See the CI failure in the commit below.

DeprecationWarning: Creating a LegacyVersion has been deprecated and will be removed in the next major release

I can reproduce the failure locally with nox.

@Jasha10
Copy link
Collaborator Author

Jasha10 commented Nov 15, 2021 •
edited

This seems to be related to pypa/setuptools#2885.
For now I'm going to pin setuptools!=59.1.0 to resolve our CI failure.

@Jasha10 Jasha10 marked this pull request as ready for review November 15, 2021 14:10
@Jasha10
Copy link
Collaborator Author

Jasha10 commented Nov 15, 2021

As of this writing, setuptools version 51.1.0 has been yanked from pypi. As such, the pin implemented by this PR is not necessary.

@Jasha10 Jasha10 closed this Nov 15, 2021
@Jasha10 Jasha10 deleted the fix_ci branch November 15, 2021 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@Jasha10