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

Pin setuptools to <64 #3771

Merged
merged 1 commit into from
Aug 16, 2022
Merged

Pin setuptools to <64 #3771

merged 1 commit into from
Aug 16, 2022

Conversation

sondreso
Copy link
Collaborator

@sondreso sondreso commented Aug 16, 2022

Issue
setuptools version 64 seems to have broken editable installs
using scikit build.

Approach
Pin setuptools

Pre review checklist

  • Added appropriate release note label
  • PR title captures the intent of the changes, and is fitting for release notes.
  • Commit history is consistent and clean, in line with the contribution guidelines.

Adding labels helps the maintainers when writing release notes. This is the list of release note labels.

setuptools version 64 seems to have broken editable installs
using scikit build.
@sondreso sondreso self-assigned this Aug 16, 2022
@sondreso sondreso added maintenance Not a bug now but could be one day, repaying technical debt release-notes:maintenance Automatically categorise as maintenance change in release notes and removed maintenance Not a bug now but could be one day, repaying technical debt labels Aug 16, 2022
@sondreso sondreso enabled auto-merge (rebase) August 16, 2022 10:54
@sondreso sondreso merged commit 89a461b into equinor:main Aug 16, 2022
@sondreso sondreso deleted the pin-setuptools branch August 16, 2022 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-notes:maintenance Automatically categorise as maintenance change in release notes
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants