fix: remove upper pin on vyper dependency

ApeWorX / ape-vyper

Vyper compiler plugin for the Ape Framework, using VVM

https://www.apeworx.io/

Apache License 2.0

24 stars 9 forks source link

fix: remove upper pin on vyper dependency #115

Closed antazoey closed 2 weeks ago

antazoey commented 3 weeks ago

What I did

trying to assist in envs where vyper is installed in 0.4

fixes: #114 fixes APE-1762

How I did it

How to verify it

Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

antazoey commented 3 weeks ago

After some discussion, going to look into actually ensuring we can flatten on 0.4...

[ ] update with response or increased support

mikeshultz commented 3 weeks ago

After some discussion, going to look into actually ensuring we can flatten on 0.4...

I haven't looked at the vyper changelog or anything, but my primary concern is vyper modules. Right now imports are just treated as interfaces and transpiled accordingly. Which is probably not what we want for modules. But most of what I'm saying is guesswork. Needs investigation.

antazoey commented 2 weeks ago

closing in favor of #118