all: fix nolint directives

Change-Id: I261c8b12e4961e6401cc4024fa5abc35b1a5efa6

private/post/message.go

@@ -49,7 +49,7 @@ func (msg *Message) Bytes() (data []byte, err error) {
 	fmt.Fprintf(&body, "Subject: %v\r\n", mime.QEncoding.Encode("utf-8", msg.Subject))
 	fmt.Fprintf(&body, "From: %s\r\n", &msg.From)
 	for _, to := range msg.To {
-		fmt.Fprintf(&body, "To: %s\r\n", &to) // nolint:scopelint
+		fmt.Fprintf(&body, "To: %s\r\n", &to) //nolint:scopelint
 	}
 	for _, recipient := range msg.ReceiptTo {
 		fmt.Fprintf(&body, "Disposition-Notification-To: <%v>\r\n", mime.QEncoding.Encode("utf-8", recipient))

satellite/audit/verifier_test.go

@@ -1044,7 +1044,8 @@ func TestAuditRepairedSegmentInExcludedCountries(t *testing.T) {
 }
 
 // getRemoteSegment returns a remote pointer its path from satellite.
-// nolint:golint
+//
+//nolint:golint
 func getRemoteSegment(
 	ctx context.Context, t *testing.T, satellite *testplanet.Satellite, projectID uuid.UUID, bucketName string,
 ) (_ metabase.Segment, key metabase.SegmentKey) {

satellite/metabase/bench_expired_test.go

@@ -95,7 +95,7 @@ func (s *expiredScenario) name() string {
 
 // run runs the specified scenario.
 //
-// nolint: scopelint // This heavily uses loop variables without goroutines, avoiding these would add lots of boilerplate.
+//nolint:scopelint // This heavily uses loop variables without goroutines, avoiding these would add lots of boilerplate.
 func (s *expiredScenario) run(ctx *testcontext.Context, b *testing.B, db *metabase.DB) {
 	if s.redundancy.IsZero() {
 		s.redundancy = storj.RedundancyScheme{

satellite/metabase/bench_test.go

@@ -53,7 +53,7 @@ func (s *scenario) name() string {
 
 // run runs the specified scenario.
 //
-// nolint: scopelint // This heavily uses loop variables without goroutines, avoiding these would add lots of boilerplate.
+//nolint:scopelint // This heavily uses loop variables without goroutines, avoiding these would add lots of boilerplate.
 func (s *scenario) run(ctx *testcontext.Context, b *testing.B, db *metabase.DB) {
 	if s.redundancy.IsZero() {
 		s.redundancy = storj.RedundancyScheme{

satellite/repair/repair_test.go

@@ -2337,7 +2337,8 @@ func TestRepairGracefullyExited(t *testing.T) {
 }
 
 // getRemoteSegment returns a remote pointer its path from satellite.
-// nolint:golint
+//
+//nolint:golint
 func getRemoteSegment(
 	ctx context.Context, t *testing.T, satellite *testplanet.Satellite, projectID uuid.UUID, bucketName string,
 ) (_ metabase.Segment, key metabase.SegmentKey) {