-
Notifications
You must be signed in to change notification settings - Fork 197
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 page for describing cairo builtins #1202
Conversation
b473943
to
bd3e467
Compare
@stoobie I finished the doc, please help me to review. Thank you very much. |
@xiaolou86 Thank you! I've forwarded this on for a technical review. Once that is done, I'll go over it and edit or make suggestions. |
@xiaolou86 Here's the tech review feedback I received:
It would be good to provide a code example of a builtin vs accomplishing the same thing in standard Cairo. |
ok, I will update the docs. |
@stoobie please help to review again. |
Please mention that you don't need to do anything to use a builtin. The library you use should automatically scan your code and use a builtin where appropriate. I'm not 100% sure if I got this right, so please wait for a confirmation from @ArielElp. Also, the only builtins that you should have on this page are the ones [isted in on the docs list in the fee page in Table 1. Amount of gas used per Cairo step or per each time a Cairo builtin is applied) |
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.
@xiaolou86 I made a couple more comments/suggestions. I asked someone more knowledgeable than me to comment as well, so please be aware that he needs to respond. You can make the minor changes I requested in the meantime.
components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/builtins.adoc
Outdated
Show resolved
Hide resolved
components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/builtins.adoc
Outdated
Show resolved
Hide resolved
components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/builtins.adoc
Outdated
Show resolved
Hide resolved
…art_Contracts/builtins.adoc Co-authored-by: Steve Goodman <[email protected]>
…art_Contracts/builtins.adoc Co-authored-by: Steve Goodman <[email protected]>
All right.
I think I have the only ones on this page, but I am not much sure. Thank you very much. |
components/Starknet/modules/architecture_and_concepts/pages/Smart_Contracts/builtins.adoc
Outdated
Show resolved
Hide resolved
…art_Contracts/builtins.adoc Co-authored-by: Steve Goodman <[email protected]>
@xiaolou86 After reviewing this with our expert, I found that we were on the wrong track. I created a new PR (#1243) with the necessary info. Feel free to review it and make suggestions or ask questions if anything is unclear. After the new PR is merged, I'll settle with you on OnlyDust. |
Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1202/documentation/ . |
4 similar comments
Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1202/documentation/ . |
Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1202/documentation/ . |
Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1202/documentation/ . |
Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1202/documentation/ . |
Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1202/documentation/ . |
2 similar comments
Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1202/documentation/ . |
Your preview build is ready! ✨ Check the following link in 1-2 minutes: https://starknet-io.github.io/starknet-docs/pr-1202/documentation/ . |
#1243 Closes this. |
Description of the Changes
Closes #1126
Add pages for describing cairo builtins
PR Preview URL
After you push a commit to this PR, a preview is built and a URL to the root of the preview appears in the comment feed.
Paste here the specific URL(s) of the content that this PR addresses.
Check List
<docs/feat/fix/chore>(optional scope): <description>
, e.g:fix: minor typos in code
This change is