-
Notifications
You must be signed in to change notification settings - Fork 5
Fixes for contracts-cairo #27
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
✅ Deploy Preview for openzeppelin-docs-v2 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
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.
@ericnordelo FYI the reason the build was failing was due to using .ts file references in the markdown which was causing some weird behaviors, likely due to some kind of race condition of compiling the ts and the markdown. Updating it to JS fixed it, hope that's ok!
Everything looks great to me! Especially love some of the API components you put together, I might try to implement that into the solidity docgen templates. If everything looks good to you too then either you or I can merge it :) |
On-going effort to make the contracts-cairo docs ready for release.