-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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: Improve documentation for ERC-4626 interface #12871
Conversation
✅ Deploy Preview for ethereumorg ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@coderabbitai review |
WalkthroughThe recent modifications to the ERC-4626 documentation involve renaming return variables in various view functions to enhance clarity. These changes make the function outputs more descriptive, aiding developers in better understanding the interface's workings. Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
@raxhvl, thank you for your contribution! I made a version more aligned with the overall style of the map image. What do you think? If everything is ok with the image, can you make the change on the PR? |
@nloureiro I love it!. A small fix: When you are done making the change, I will add that to the PR. Thank you for making this look good, oh design whisperer! |
Great! @nloureiro assigning to you for design review, @ me when ready and I can do a final sweep |
oh, yes, my bad... should be good now |
@nloureiro @wackerow All clear! |
@raxhvl sorry... I saw it on a good display and realized I should have exported 2x the size. |
@nloureiro. Done! Quick heads up the resolution has increased but the image size is actually smaller 70 kb vs 77kb from earlier. |
yes, no worries. 77kb is still a pretty small size for this. all is good on my end. thank you for your contribution. |
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.
Nice, LGTM! Thanks @nloureiro and @raxhvl!
@all-contributors can you add @raxhvl for content |
I've put up a pull request to add @raxhvl! 🎉 |
Congrats, your important contribution to this open-source project has earned you a GitPOAP! Be sure to join the Ethereum.org discord if you are interested in contributing further to the project or have any questions for the team. GitPOAP: 2024 Ethereum.org Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
Description
Related Issue
closes #12870
Summary by CodeRabbit