Skip to content

Commit d37bb9a

Browse files
author
Bryan C. Mills
committed
devapp/owners: add owners for build, debug, gccgo, tools, and go/src/cmd/asm
These components have had recent activity in the issue tracker, so it's helpful to add explicit owners, even if their primary role as such is to dispatch issues to more-specific owners who are not yet listed. Updates golang/go#27586 Change-Id: Ib77b6d1d02b49b9709f1f58797acf2b64ce95e48 Reviewed-on: https://go-review.googlesource.com/c/build/+/172278 Run-TryBot: Bryan C. Mills <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]>
1 parent 154f152 commit d37bb9a

File tree

1 file changed

+28
-5
lines changed

1 file changed

+28
-5
lines changed

devapp/owners/table.go

Lines changed: 28 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,14 @@ var entries = map[string]*Entry{
7676
Primary: []Owner{cherryyz},
7777
},
7878

79+
"build": {
80+
Primary: []Owner{dmitshur, bradfitz, andybons},
81+
},
82+
"build/maintner/cmd/maintserve": {
83+
Primary: []Owner{dmitshur},
84+
Secondary: []Owner{andybons},
85+
},
86+
7987
"crypto": {
8088
Primary: []Owner{filippo},
8189
Secondary: []Owner{agl},
@@ -85,6 +93,15 @@ var entries = map[string]*Entry{
8593
Secondary: []Owner{filippo},
8694
},
8795

96+
"debug": {
97+
Secondary: []Owner{hyangah, khr},
98+
},
99+
100+
"gccgo": {
101+
Primary: []Owner{iant},
102+
Secondary: []Owner{thanm, cherryyz},
103+
},
104+
88105
"go/src/archive/tar": {
89106
Primary: []Owner{joetsai},
90107
},
@@ -100,6 +117,9 @@ var entries = map[string]*Entry{
100117
Primary: []Owner{},
101118
Secondary: []Owner{bradfitz, iant},
102119
},
120+
"go/src/cmd/asm": {
121+
Primary: []Owner{cherryyz},
122+
},
103123
"go/src/cmd/compile": {
104124
Primary: []Owner{khr, gri},
105125
Secondary: []Owner{josharian, mdempsky, martisch},
@@ -288,6 +308,11 @@ var entries = map[string]*Entry{
288308
"go/src/go/internal/gcimporter": {
289309
Primary: []Owner{gri},
290310
},
311+
// go/packages doesn't exist yet, but x/tools/go/packages has been proposed to
312+
// move there and many issues already refer to the new path.
313+
"go/src/go/packages": {
314+
Primary: []Owner{matloob},
315+
},
291316
"go/src/go/parser": {
292317
Primary: []Owner{gri},
293318
},
@@ -537,11 +562,6 @@ var entries = map[string]*Entry{
537562
Primary: []Owner{gri},
538563
},
539564

540-
"build/maintner/cmd/maintserve": {
541-
Primary: []Owner{dmitshur},
542-
Secondary: []Owner{andybons},
543-
},
544-
545565
"gofrontend": {
546566
Primary: []Owner{iant},
547567
Secondary: []Owner{thanm},
@@ -627,6 +647,9 @@ var entries = map[string]*Entry{
627647
"tools/internal/lsp": {
628648
Primary: []Owner{rstambler, iancottrell},
629649
},
650+
"tools": {
651+
Primary: []Owner{iancottrell},
652+
},
630653
"playground": {
631654
Secondary: []Owner{ysmolsky},
632655
},

0 commit comments

Comments
 (0)