-
Notifications
You must be signed in to change notification settings - Fork 189
feat: add support for codeOf T
feature
#1948
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's also refactor the Jetton contracts to use codeOf
:
...
// Owner may be any address, what matters here is the code, not data.
jettonWalletCode: initOf JettonWallet(myAddress(), myAddress()).code
...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome stuff! Can you take a stub at documenting codeOf
in the Tact docs please? It would be nice to mention the "algebraic laws" there (the relationships between initOf
, myCode
and codeOf
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doc good rest ok
P.S.: The grammar was intentionally butchered for meme purposes. Thank you for your attention.
I'd also try to reuse tests I've written for InitOf, so they would use codeOf. (Or added more) |
Gas usage has just become higher for some reason |
codeOf T
expression tree-sitter-tact#50codeOf
tact-language-server#356codeOf
support tact-sublime#31Benchmark
codeOf Foo
vsinitOf Foo().code
:Issue
Closes #1216.
Checklist
docs/
and made the build locally