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

feat: add flag to install_vyper() for skipping validation #33

Merged
merged 6 commits into from
Jan 27, 2025

Conversation

antazoey
Copy link
Contributor

@antazoey antazoey commented Jan 7, 2025

What I did

Related issue: #32

How I did it

Add a bool kwarg and honor it.

How to verify it

This is on an arm64 device (mac):

In [2]: vvm.install.install_vyper("0.3.7", validate=False)
Out[2]: <Version('0.3.7')>

obviously it still does not work but at least I have it, which helps me

Checklist

  • I have confirmed that my PR passes all linting checks
  • I have included test cases
  • I have updated the documentation (README.md)
  • I have added an entry to the changelog

@charles-cooper charles-cooper merged commit 8c5b8bf into vyperlang:master Jan 27, 2025
9 of 10 checks passed
@charles-cooper
Copy link
Member

thanks!

@antazoey antazoey deleted the feat/skip-validate branch January 27, 2025 17:42
@antazoey
Copy link
Contributor Author

I am not sure why that windows test was failing still tho

@antazoey antazoey restored the feat/skip-validate branch January 27, 2025 17:42
@antazoey antazoey deleted the feat/skip-validate branch January 27, 2025 17:43
@charles-cooper
Copy link
Member

it's an existing issue on windows, not a regression

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.

3 participants