Skip to content

Conversation

@bryevdv
Copy link
Contributor

@bryevdv bryevdv commented May 22, 2025

No description provided.

@bryevdv bryevdv requested a review from Jacobfaib May 22, 2025 16:27

[build-system]
requires = ["rich", "setuptools", "setuptools-scm"]
requires = ["rich", "setuptools>80", "setuptools-scm"]
Copy link
Contributor

@Jacobfaib Jacobfaib May 22, 2025

Choose a reason for hiding this comment

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

>80 or >=80?

Copy link
Contributor Author

@bryevdv bryevdv May 22, 2025

Choose a reason for hiding this comment

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

There are several "80" versions, so I didn't think it mattered at this point, really.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it's instructive for a future reader to know exactly what version introduced what we need. >80 could mean 81, or 80.0.1

Copy link
Contributor

Choose a reason for hiding this comment

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

In any case, not a huge deal, I won't hold up the PR over this


[build-system]
requires = ["rich", "setuptools", "setuptools-scm"]
requires = ["rich", "setuptools>80", "setuptools-scm"]
Copy link
Contributor

Choose a reason for hiding this comment

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

In any case, not a huge deal, I won't hold up the PR over this

@bryevdv bryevdv merged commit 2deb3cd into nv-legate:main May 22, 2025
4 checks passed
@bryevdv bryevdv deleted the bv/setuptools-version branch May 22, 2025 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants