-
Notifications
You must be signed in to change notification settings - Fork 7
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
Golang docs #442
Golang docs #442
Conversation
Deploying documentation with Cloudflare Pages
|
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.
Left some minor comments, from my side the code looks good!
|
||
Request-response calls are requests where the client waits for the response. | ||
|
||
<CH.Scrollycoding className={"single-item"}> |
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.
there is a problem with tab here, not sure what
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.
not sure what you mean
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.
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.
I think the checking of the go code snippets still needs to be added to the github workflows as well |
And we should add Go icons to the overview page here: https://golang.documentation-beg.pages.dev/ |
@slinkydeveloper go uses tabs, i will check how to get our code blocks to display them tighter |
Co-authored-by: Francesco Guardiani <[email protected]>
The best way to review this is just to look at the preview