Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(53)

Issue 112650043: code review 112650043: doc: drop schema from links that are known to support HTTPS (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 9 months ago by adg
Modified:
9 years, 9 months ago
Reviewers:
gobot, bradfitz
CC:
golang-codereviews, bradfitz
Visibility:
Public.

Description

doc: drop scheme from links that are known to support HTTPS golang.org now serves HTTPS with a valid cert, so it's reasonable that users should click through to the HTTPS versions of *.golang.org and other known sites.

Patch Set 1 #

Patch Set 2 : diff -r 32ca51fbc582 https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+124 lines, -124 lines) Patch
M doc/cmd.html View 2 chunks +3 lines, -3 lines 0 comments Download
M doc/code.html View 6 chunks +8 lines, -8 lines 0 comments Download
M doc/contrib.html View 3 chunks +5 lines, -5 lines 0 comments Download
M doc/devel/release.html View 16 chunks +33 lines, -33 lines 0 comments Download
M doc/devel/weekly.html View 1 chunk +1 line, -1 line 0 comments Download
M doc/docs.html View 3 chunks +8 lines, -8 lines 0 comments Download
M doc/effective_go.html View 3 chunks +4 lines, -4 lines 0 comments Download
M doc/gccgo_contribute.html View 3 chunks +3 lines, -3 lines 0 comments Download
M doc/go1.html View 6 chunks +6 lines, -6 lines 0 comments Download
M doc/go1.1.html View 1 chunk +1 line, -1 line 0 comments Download
M doc/go1.2.html View 5 chunks +5 lines, -5 lines 0 comments Download
M doc/go1.3.html View 7 chunks +7 lines, -7 lines 0 comments Download
M doc/go1compat.html View 1 chunk +1 line, -1 line 0 comments Download
M doc/go_faq.html View 16 chunks +19 lines, -19 lines 0 comments Download
M doc/go_spec.html View 1 chunk +1 line, -1 line 0 comments Download
M doc/help.html View 2 chunks +4 lines, -4 lines 0 comments Download
M doc/install.html View 1 chunk +2 lines, -2 lines 0 comments Download
M doc/install-source.html View 4 chunks +7 lines, -7 lines 0 comments Download
M doc/root.html View 4 chunks +4 lines, -4 lines 0 comments Download
M doc/tos.html View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4
adg
Hello golang-codereviews@googlegroups.com, I'd like you to review this change to https://code.google.com/p/go
9 years, 9 months ago (2014-07-24 06:12:45 UTC) #1
bradfitz
LGTM On Wed, Jul 23, 2014 at 11:12 PM, <adg@golang.org> wrote: > Reviewers: golang-codereviews, > ...
9 years, 9 months ago (2014-07-24 15:54:46 UTC) #2
adg
*** Submitted as https://code.google.com/p/go/source/detail?r=bdac8d858e74 *** doc: drop scheme from links that are known to support ...
9 years, 9 months ago (2014-07-25 00:28:31 UTC) #3
gobot
9 years, 9 months ago (2014-07-25 01:36:08 UTC) #4
Message was sent while issue was closed.
This CL appears to have broken the netbsd-amd64-bsiegert builder.
See http://build.golang.org/log/da6ae804f02d12ba482075311e4b693b60053fe3
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b