8000 docs: fix registry code example by sagikazarmark · Pull Request #126 · go-sprout/sprout · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

docs: fix registry code example #126

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 4, 2025
Merged

docs: fix registry code example #126

merged 2 commits into from
Jun 4, 2025

Conversation

sagikazarmark
Copy link
Contributor
@sagikazarmark sagikazarmark commented May 12, 2025 •
edited
Loading

Description

Fix the registry code example in the documentation

Changes

  • fix function signature
  • return nil error by default to make the code compile

Checklist

  • I have read the CONTRIBUTING.md document.
  • My code follows the code style of this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have updated the documentation accordingly.
  • This change requires a change to the documentation on the website.

Additional Information

@sagikazarmark
docs: fix registry code example
f53d1b0 
Signed-off-by: Márk Sági-Kazár <mark.sagikazar@gmail.com>
42atomys
42atomys approved these changes Jun 4, 2025
View reviewed changes
Copy link
Member
@42atomys 42atomys left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, thanks for this typo fix 🌱

@42atomys 42atomys added the aspect/documentation 📚 Improvements or additions to documentation label Jun 4, 2025
@42atomys 42atomys enabled auto-merge (squash) June 4, 2025 22:09
@42atomys 42atomys disabled auto-merge June 4, 2025 22:11
@42atomys 42atomys merged commit 41e9247 into go-sprout:main Jun 4, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@42atomys 42atomys 42atomys approved these changes

Assignees
No one assigned
Labels
aspect/documentation 📚 Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sagikazarmark @42atomys
0