Skip to content

Commit 6f7df8f

Browse files
committed
devapp/owners: add mvdan as secondary to cmd/gofmt
I've contributed sporadically over the years, and more recently by experimenting externally in gofumpt. With a CL in-flight to make gofmt format files in parallel, it feels like it's time to make this a bit more official. The main important bits are cmd/gofmt, the tool, and go/printer, the guts of the tool. go/format is pulled in as a library version of cmd/gofmt. Note that cmd/gofmt wasn't in the table, but it feels like it should be there, as it's pretty different from the other cmd/... tools. Change-Id: Ia3455707105b8ab1fcd35799b592de85a9313655 Reviewed-on: https://go-review.googlesource.com/c/build/+/350550 Trust: Daniel Martí <[email protected]> Reviewed-by: Robert Griesemer <[email protected]>
1 parent 35ff7b4 commit 6f7df8f

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

devapp/owners/table.go

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,10 @@ var entries = map[string]*Entry{
174174
Primary: []Owner{bcmills, jayconrod, matloob},
175175
Secondary: []Owner{rsc, iant},
176176
},
177+
"go/src/cmd/gofmt": {
178+
Primary: []Owner{gri},
179+
Secondary: []Owner{mvdan},
180+
},
177181
"go/src/cmd/internal/obj/wasm": wasmOwners,
178182
"go/src/cmd/link": {
179183
Primary: []Owner{cherryyz, rsc, iant},
@@ -293,7 +297,8 @@ var entries = map[string]*Entry{
293297
Secondary: []Owner{agnivade},
294298
},
295299
"go/src/go/format": {
296-
Primary: []Owner{gri},
300+
Primary: []Owner{gri},
301+
Secondary: []Owner{mvdan},
297302
},
298303
"go/src/go/importer": {
299304
Primary: []Owner{gri, adonovan},
@@ -313,7 +318,8 @@ var entries = map[string]*Entry{
313318
Primary: []Owner{gri},
314319
},
315320
"go/src/go/printer": {
316-
Primary: []Owner{gri},
321+
Primary: []Owner{gri},
322+
Secondary: []Owner{mvdan},
317323
},
318324
"go/src/go/scanner": {
319325
Primary: []Owner{gri},

0 commit comments

Comments
 (0)