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

Docs: explainer about macros #200

Closed
jzaki opened this issue Mar 22, 2024 · 4 comments · Fixed by AztecProtocol/aztec-packages#7172
Closed

Docs: explainer about macros #200

jzaki opened this issue Mar 22, 2024 · 4 comments · Fixed by AztecProtocol/aztec-packages#7172
Assignees

Comments

@jzaki
Copy link
Contributor

jzaki commented Mar 22, 2024

blocked until new docs structure

@jzaki
Copy link
Contributor Author

jzaki commented Mar 22, 2024

Related slack thread

  1. From what I can see the name _initialize doesn't have special meaning, and is just convention (like in bb and wasm) ?
  2. The contract is checked for having a macro defining initializer, if so, then every function will default to disallowing calls before the/an initializer function has been called?
  • noinitcheck overrides this check and will call it without needing the/a preceding initializer function being called?
  • Multiple functions can be defined as initializer, and any one will allow other functions to be called (that don't have noinitcheck)?
  1. internal simply does what it says on the tin here?

and doc

@critesjosh
Copy link
Collaborator

Do we need to wait until there is a new docs structure to work on this?

This was referenced Apr 15, 2024
@catmcgee
Copy link
Collaborator

Do we need to wait until there is a new docs structure to work on this?

No, especially because docs structure won't be started on for another 3 weeks. i'll take this as i'm assigned

@catmcgee
Copy link
Collaborator

add Grego as a reviewer

catmcgee added a commit to AztecProtocol/aztec-packages that referenced this issue Jul 2, 2024
I had already written a lot of this before so just adding to the other
macros since then

Closes AztecProtocol/dev-rel#200

---------

Co-authored-by: James Zaki <[email protected]>
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 a pull request may close this issue.

3 participants