Skip to content
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

Update Go documentation #440

Merged
merged 6 commits into from Jun 17, 2022
Merged

Update Go documentation #440

merged 6 commits into from Jun 17, 2022

Conversation

alexsaezm
Copy link
Contributor

Few things have changed and modules are now a critical part of the process.

I also changed the github.com/gorilla/context project for github.com/go-delve/delve because it is deprecated and doesn't work anymore. Delve is the de facto debugger so seems like a good example.

Also, change the project in the example because it is a deprecated
project and doesn't work properly
Copy link
Contributor

@jackorp jackorp left a comment

Choose a reason for hiding this comment

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

Thanks! Mostly LGTM, please address the comments.

tech/languages/go/go-programs.md Outdated Show resolved Hide resolved
tech/languages/go/go-packages.md Outdated Show resolved Hide resolved
tech/languages/go/go-packages.md Outdated Show resolved Hide resolved
@alexsaezm
Copy link
Contributor Author

I'm gonna change my email because GH web interface used my personal one.

alexsaezm and others added 3 commits June 17, 2022 13:49
Co-authored-by: Jaroslav Prokop <33065078+jackorp@users.noreply.github.com>
Co-authored-by: Jaroslav Prokop <33065078+jackorp@users.noreply.github.com>
Co-authored-by: Jaroslav Prokop <33065078+jackorp@users.noreply.github.com>
@alexsaezm
Copy link
Contributor Author

I'm gonna change my email because GH web interface used my personal one.

Done.

@alexsaezm alexsaezm requested a review from jackorp June 17, 2022 11:58
@jackorp
Copy link
Contributor

jackorp commented Jun 17, 2022

Thanks! Merging.

@jackorp jackorp merged commit 4da7931 into developer-portal:master Jun 17, 2022
@xsuchy
Copy link
Contributor

xsuchy commented Jun 17, 2022

Thank you all.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants