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(deps): upgrade packaging #2557

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

fubuloubu
Copy link
Member

What I did

I guess setuptools upgraded there pin and it screwed us:
https://github.com/ApeWorX/ape/actions/runs/13958350355/job/39075553268

How I did it

How to verify it

Checklist

  • All changes are completed
  • Change is covered in tests
  • Documentation is complete

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@fubuloubu fubuloubu requested a review from antazoey March 20, 2025 01:43
@fubuloubu fubuloubu enabled auto-merge (squash) March 20, 2025 01:44
antazoey
antazoey previously approved these changes Mar 20, 2025
@@ -94,7 +94,7 @@
"cchecksum>=0.0.3,<1",
# Pandas peer-dep: Numpy 2.0 causes issues for some users.
"numpy<2",
"packaging>=23.0,<24",
"packaging>=24.2,<25",
Copy link
Member

Choose a reason for hiding this comment

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

We have to fix in some downstream plugins before this will succeed: https://github.com/ApeWorX/ape-hardhat/blob/main/setup.py#L81

Copy link
Member

Choose a reason for hiding this comment

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

Actually... This is a bit of a circular dependency. The container depends on both core Ape and plugins. Plugins depend on core ape. This why I think recommened_plugins type flows don't make much sense.

The only solution to merge with with a failing solution and then update the plugin and then the next update of core ape will have it all fixed.

@antazoey
Copy link
Member

@fubuloubu can we merge this? ill fix the plugins

@fubuloubu
Copy link
Member Author

@fubuloubu can we merge this? ill fix the plugins

Yes, sounds more complex than I expected, thank you for taking this up!

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.

None yet

2 participants