Switch to godoc-style comments. (#4760)

* Switch to godoc-style comments.

staticcheck cleanup: https://staticcheck.io/docs/checks#ST1021

* Revert spurious diff.

core/objects.go

@@ -577,7 +577,7 @@ type Order struct {
 	Status            AcmeStatus
 }
 
-// SCTDER is a convenience type
+// SCTDERs is a convenience type
 type SCTDERs [][]byte
 
 // CertDER is a convenience type that helps differentiate what the

db/mocks.go

@@ -14,22 +14,22 @@ import (
 // that a context has already been applied to the relevant DbMap or
 // Transaction object.
 
-// A `dbOneSelector` is anything that provides a `SelectOne` function.
+// A OneSelector is anything that provides a `SelectOne` function.
 type OneSelector interface {
 	SelectOne(interface{}, string, ...interface{}) error
 }
 
-// A `Selector` is anything that provides a `Select` function.
+// A Selector is anything that provides a `Select` function.
 type Selector interface {
 	Select(interface{}, string, ...interface{}) ([]interface{}, error)
 }
 
-// A `Inserter` is anything that provides an `Insert` function
+// A Inserter is anything that provides an `Insert` function
 type Inserter interface {
 	Insert(list ...interface{}) error
 }
 
-// A `Execer` is anything that provides an `Exec` function
+// A Execer is anything that provides an `Exec` function
 type Execer interface {
 	Exec(string, ...interface{}) (sql.Result, error)
 }

mocks/mocks.go

@@ -749,7 +749,7 @@ func (m *Mailer) Connect() error {
 	return nil
 }
 
-// mockSAWithFailedChallenges is a mocks.StorageAuthority that has
+// SAWithFailedChallenges is a mocks.StorageAuthority that has
 // a `GetAuthorization` implementation that can return authorizations with
 // failed challenges.
 type SAWithFailedChallenges struct {

test/load-generator/boulder-calls.go

@@ -41,8 +41,8 @@ var (
 	}
 )
 
-// It's awkward to work with core.Order or corepb.Order when the API returns
+// OrderJSON is used because it's awkward to work with core.Order or corepb.Order
-// a different object than either of these types can represent without
+// when the API returns a different object than either of these types can represent without
 // converting field values. The WFE uses an unexported `orderJSON` type for the
 // API results that contain an order. We duplicate it here instead of moving it
 // somewhere exported for this one utility.