-
Notifications
You must be signed in to change notification settings - Fork 75
Add Golang to Languages and Frameworks #384
Conversation
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.
Great initial Go documentation! Many thanks for writing it. 👍
Added a few comments, as usual. 😛
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.
Starting to look very good! Many thanks for writing this guide.
A few more comments, because I couldn't resist. 😅
Signed-off-by: Sean Hellum <[email protected]>
Co-Authored-By: Jan Keromnes <[email protected]>
Co-Authored-By: Jan Keromnes <[email protected]>
Co-Authored-By: Jan Keromnes <[email protected]>
Co-Authored-By: Jan Keromnes <[email protected]>
Co-Authored-By: Jan Keromnes <[email protected]>
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.
This is looking almost final! Many thanks for writing a Go guide so fast.
I added a small comment about a confusing phrasing; a confusing title; and I made a general comment about the tone of this article and who it should be written for. 🙂
All-in-all, I think we'll be able to merge this soon. Thanks again! 💯
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.
Looking good! Ready to merge, except for a few tiny remaining details.
Co-Authored-By: Jan Keromnes <[email protected]>
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.
Perfect, many thanks! I think we can merge this as an initial version, and then iterate on it later (e.g. when Go debugging works 😊).
Before merging though, let's wait for Netlify's deployment, to give a quick look at the rendered page (and maybe catch any potentially remaining markdown bugs).
Looks good! |
@jankeromnes the netlify deployment looks good |
Add Golang to languages and frameworks
see #383