Skip to content

updated and extended snippets #11

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

Merged
merged 2 commits into from
Jan 20, 2022
Merged

Conversation

Doc-Pixel
Copy link
Contributor

Hi!

I've updated the snippets file to reflect Vyper 0.3.2. Current stablevyper version is 0.3.1 but I've added some 0.3.2 things as well, Mostly the DynArray syntax.

".source.solidity": {

"mapping declaration":{
".source.solidity": {
Copy link
Owner

Choose a reason for hiding this comment

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

huh, I guess I forgot to change this when I copied it from my solidity extension 😅. I think it should be safe to change this to .source.vyper 🙏

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would you like me to do that and update / resubmit the PR?

Copy link
Owner

Choose a reason for hiding this comment

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

that'd be great 🙌

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've added a commit with that change. I see it in the commits tab here so I think it's part of the PR now?

@tintinweb tintinweb merged commit 507c874 into tintinweb:master Jan 20, 2022
@tintinweb
Copy link
Owner

thank you @DataBeast-IT 🙌

@tintinweb
Copy link
Owner

published.

@ghost
Copy link

ghost commented May 18, 2022

[Compiler Error] vyper.exceptions.UnknownType: No builtin or user-defined type named 'DynArray'
contract , line 8:2
7
---> 8 x:DynArray[uint256, 1]
---------^
9 insured: HashMap[address,uint256]

Compilation of failed. See above.

if published why not suported?

@Doc-Pixel
Copy link
Contributor Author

Because you are on a vyper version lower than 0.3.2.
Also the vs code extension does not compile the smart contract. It calls vyper, which is a different project all together 😎

If you update Vyper and set the version to at least 0.3.2 I am confident it will work

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