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

chore: update vyper imports #46

Merged
merged 22 commits into from
Mar 7, 2023
Merged

Conversation

tserg
Copy link
Contributor

@tserg tserg commented Nov 28, 2022

Update imports from Vyper as a result of vyperlang/vyper#2974, vyperlang/vyper#3182, vyperlang/vyper#3209

Also fixes test path and updated README.

@tserg tserg marked this pull request as ready for review November 28, 2022 10:44
Copy link
Member

@charles-cooper charles-cooper left a comment

Choose a reason for hiding this comment

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

looks good but holding off on merging this as a lot of other changes to the semantics pass are coming up for vyper

pyproject.toml Outdated Show resolved Hide resolved
Copy link
Member

@charles-cooper charles-cooper left a comment

Choose a reason for hiding this comment

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

thanks! saved me a bunch of work!

@charles-cooper
Copy link
Member

looks like there's a couple merge conflicts though - could you please fix?

@tserg
Copy link
Contributor Author

tserg commented Mar 7, 2023

looks like there's a couple merge conflicts though - could you please fix?

I have fixed the merge conflict. Oddly, CI is now failing because of missing packages. I am still figuring out why.

@tserg tserg requested a review from charles-cooper March 7, 2023 10:34
@charles-cooper charles-cooper merged commit 1e67558 into vyperlang:master Mar 7, 2023
@tserg tserg deleted the chore/bump_vyper branch March 8, 2023 01:06
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