-
Notifications
You must be signed in to change notification settings - Fork 17
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(go): comments are enhanced to provide better developer experience #1536
Conversation
✅ Deploy Preview for api-clients-automation ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✗ The generated branch has been deleted.If the PR has been merged, you can check the generated code on the |
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.
LGTM!
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.
Huge improvements, gg!
algolia/api-clients-automation#1536 Co-authored-by: Mehmet Ali Gok <33124154+mehmetaligok@users.noreply.github.com>
algolia/api-clients-automation#1536 Co-authored-by: Mehmet Ali Gok <33124154+mehmetaligok@users.noreply.github.com>
algolia/api-clients-automation#1536 Co-authored-by: Mehmet Ali Gok <33124154+mehmetaligok@users.noreply.github.com>
algolia/api-clients-automation#1536 Co-authored-by: Mehmet Ali Gok <33124154+mehmetaligok@users.noreply.github.com>
algolia/api-clients-automation#1536 Co-authored-by: Mehmet Ali Gok <33124154+mehmetaligok@users.noreply.github.com>
🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/APIC-665
To improve the developer experience, we need better documentation on our code so the users will not need to switch between external documentation and code again and again.
Changes included:
🧪 Test
Nothing changed functionally, generated commit can be checked to see examples of the comments.