search

vyperlang / vyper

Pythonic Smart Contract Language for the EVM
https://vyperlang.org
Other
4.83k stars 789 forks source link

refactor[ux]: remove deprecated `VyperNode` properties #3999

Closed tserg closed 3 months ago

tserg commented 3 months ago

What I did

Depends on #3998.

Remove deprecated properties from Num and Str vyper nodes following in the spirit of the equivalent deprecated python AST classes.

How I did it

How to verify it

Commit message

This commit removes the `n` property from `Num` vyper node and the `s`
property from `Str` vyper node as they have been deprecated in the
python AST (and it is clearer to use the `value` property anyways)

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

charles-cooper commented 3 months ago

@tserg this is ready for review right?

tserg commented 3 months ago

@tserg this is ready for review right?

Yes, just pulled the latest master.