-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
blog: fix code.google.com links #23688
Comments
is the blog part of this repo? are you accepting pr's ? |
Yes: https://github.com/golang/blog/ We don't accept PRs via GitHub ... yet. Soon. Currently you need to use Gerrit, but wait maybe a week and we should accept PRs. |
@bradfitz, those links in the article point to sources which were deleted in this commit:
Does it makes sense to point to different but actual examples in the repo? |
Why don't you just change the links to point to source browser URLs linking into old branches? |
Change https://golang.org/cl/94235 mentions this issue: |
Alright, done. I have fixed all other broken links I could find. |
Deprecated links to code.google.com were replaced with relevant links. Fixes golang/go#23688 Change-Id: I6f97ce072258a3a319df22d1df5773726ad5b801 Reviewed-on: https://go-review.googlesource.com/94235 Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> X-Blog-Commit: b75d504c8a09e20834f367977a6dc71353d8131a
Deprecated links to code.google.com were replaced with relevant links. Fixes golang/go#23688 Change-Id: I6f97ce072258a3a319df22d1df5773726ad5b801 Reviewed-on: https://go-review.googlesource.com/94235 Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> X-Blog-Commit: b75d504c8a09e20834f367977a6dc71353d8131a
Source: https://blog.golang.org/introducing-gofix
Summary: The links in the article above (paragraph 4) redirect the user to the Go github repository landing page.
See also: #8692
The text was updated successfully, but these errors were encountered: