Skip to content

Upgrade setuptools.#1309

Merged
bsowell merged 2 commits into
mainfrom
upgrade_setuptools
May 20, 2025
Merged

Upgrade setuptools.#1309
bsowell merged 2 commits into
mainfrom
upgrade_setuptools

Conversation

@bsowell
Copy link
Copy Markdown
Contributor

@bsowell bsowell commented May 20, 2025

@bsowell bsowell requested a review from karanataryn May 20, 2025 19:27
grpcio = "^1.60.0"
cbor2 = "^5.6.0"
sycamore-ai = "^0.1.13"
protobuf = "4.25.3" # See poetry-lock/pyproject.toml
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are we sure we don't need this anymore? Would rather delete it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We don't use remote-processors anymore. poetry-lock was just broken in this project with this. I can delete it, but I confess I don't know whether there will be issues.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ok got it. I'd suggest adding a comment stating that at least.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I removed it. Don't think it matters much either way since I don't think anyone knows or is using this code.

Comment thread lib/sycamore/pyproject.toml Outdated
overrides = "^7.7.0"
structlog = "^24.2.0"
setuptools = ">70.0.0"
setuptools = ">78.1.1"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nit: > or >= ? Looks like the latter

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure.

@bsowell bsowell merged commit 2109732 into main May 20, 2025
11 of 15 checks passed
@bsowell bsowell deleted the upgrade_setuptools branch May 20, 2025 21:30
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