net/http: golint var and function name fixes #27974
Conversation
Currently there is no example for http.Post in the documentation. This adds an example. Change-Id: I4ef827381cb8fc601dcd6fd73755b097be1905d2
Resolves `client.go:217:39: error strings should not be capitalized or end with punctuation or a newline` detected by `golint` Updates #21779 Change-Id: I59767dd7b09d2f12a8f72d79c103dbb0ba908d5d
Resolves `cookie.go:33:2: struct field HttpOnly should be HTTPOnly` detected by `golint` Updates #21779 Change-Id: I4813344021791e92727bd7db020291608e7290cf
Resolves various var naming issues detected by `golint` Updates #21779 Change-Id: Idbfdcd3ecc15d12df97e7a506a295654c1d626cc
Resolves `request.go:1321:19: method wantsHttp10KeepAlive should be wantsHTTP10KeepAlive` detected by `golint` Change-Id: I6908a46f393118d212069b61e7f79fe7e47c892b
Resolves various var naming issues detected by `golint` Updates #21779 Change-Id: I70d0cf1356dd25b74a290c9a01602b97a247a348
googlebot
added
the
cla: yes
|
This PR (HEAD: eb144bf) has been imported to Gerrit for code review. Please visit https://go-review.googlesource.com/c/go/+/139097 to see it. Tip: You can toggle comments from me using the |
|
Message from Gerrit User 5065: Patch Set 1: Code-Review-2 (1 comment) This breaks the API promise and makes a bunch of unnecessary changes to internal stuff. I don't see anything in here we need. Please don’t reply on this GitHub thread. Visit golang.org/cl/139097. |
|
Message from Gerrit User 28970: Patch Set 1: Yikes, ok. Is it proper procedure to close the PR out on Github then? Please don’t reply on this GitHub thread. Visit golang.org/cl/139097. |
|
Message from Gerrit User 5065: Patch Set 1: Yeah, you can just close the PR on GitHub. Please don’t reply on this GitHub thread. Visit golang.org/cl/139097. |
|
This PR is being closed because golang.org/cl/139097 has been abandoned. |
Various fixes for linting issues found by golint in the net/http package
Updates #21779