Skip to content
This repository was archived by the owner on Mar 14, 2023. It is now read-only.

Commit 9e4fb8d

Browse files
jackh726jackh726
committed
Add groups from expert map
1 parent 62b1531 commit 9e4fb8d

File tree

3 files changed

+110
-33
lines changed

3 files changed

+110
-33
lines changed

‎highfive/configs/rust-lang/rust.json

Lines changed: 90 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@
1616
"@LeSeulArtichaut",
1717
"@cjgillot"
1818
],
19+
"compiler": [
20+
"compiler-team",
21+
"compiler-team-contributors"
22+
],
1923
"libs": [
2024
"@dtolnay",
2125
"@joshtriplett",
@@ -33,42 +37,95 @@
3337
"@GuillaumeGomez",
3438
"@ollie27",
3539
"@CraftSpider"
40+
],
41+
"query-system": [
42+
"@cjgillot"
43+
],
44+
"incremental": [
45+
"@michaelwoerister",
46+
"@wesleywiser"
47+
],
48+
"typeck": [],
49+
"diagnostics": [
50+
"@davidtwco",
51+
"@estebank",
52+
"@oli-obk"
53+
],
54+
"parser": [
55+
"@davidtwco",
56+
"@estebank",
57+
"@petrochenkov"
58+
],
59+
"lexer": [
60+
"@petrochenkov"
61+
],
62+
"mir": [
63+
"@davidtwco",
64+
"@oli-obk"
65+
],
66+
"mir-opt": [
67+
"@nagisa",
68+
"@oli-obk",
69+
"@wesleywiser"
70+
],
71+
"traits": [
72+
"@jackh726",
73+
"@matthewjasper"
74+
],
75+
"borrowck": [
76+
"@davidtwco",
77+
"@matthewjasper",
78+
"@pnkfelix"
3679
]
3780
},
3881
"dirs": {
39-
".github/workflows": ["infra-ci"],
40-
"Cargo.lock": ["@Mark-Simulacrum"],
41-
"Cargo.toml": ["@Mark-Simulacrum"],
42-
"compiler": ["compiler-team", "compiler-team-contributors"],
43-
"compiler/rustc_apfloat": ["@eddyb"],
44-
"compiler/rustc_llvm": ["@cuviper"],
45-
"library/alloc": ["libs"],
46-
"library/core": ["libs", "@scottmcm"],
47-
"library/panic_abort": ["libs"],
48-
"library/panic_unwind": ["libs"],
49-
"library/proc_macro": ["@petrochenkov"],
50-
"library/std": ["libs"],
51-
"library/stdarch": ["libs"],
52-
"library/term": ["libs"],
53-
"library/test": ["libs"],
54-
"src/bootstrap": ["@Mark-Simulacrum"],
55-
"src/build_helper": ["@Mark-Simulacrum"],
56-
"src/ci": ["infra-ci"],
57-
"src/doc": ["doc"],
58-
"src/etc": ["@Mark-Simulacrum"],
59-
"src/librustdoc": ["rustdoc"],
60-
"src/llvm-project": ["@cuviper"],
61-
"src/stage0.txt": ["@Mark-Simulacrum"],
62-
"src/tools/cargo": ["@ehuss", "@joshtriplett"],
63-
"src/tools/compiletest": ["@Mark-Simulacrum", "@Centril"],
64-
"src/tools/linkchecker": ["@ehuss"],
65-
"src/tools/rust-installer": ["@Mark-Simulacrum"],
66-
"src/tools/rustbook": ["doc", "@ehuss"],
67-
"src/tools/rustdoc": ["rustdoc"],
68-
"src/tools/rustdoc-js": ["rustdoc"],
69-
"src/tools/rustdoc-js-std": ["rustdoc"],
70-
"src/tools/rustdoc-themes": ["rustdoc"],
71-
"src/tools/tidy": ["@Mark-Simulacrum"]
82+
".github/workflows": ["infra-ci"],
83+
"Cargo.lock": ["@Mark-Simulacrum"],
84+
"Cargo.toml": ["@Mark-Simulacrum"],
85+
"compiler": ["compiler"],
86+
"compiler/rustc_apfloat": ["@eddyb"],
87+
"compiler/rustc_ast/": ["compiler", "parser"],
88+
"compiler/rustc_lexer": ["compiler", "lexer"],
89+
"compiler/rustc_llvm": ["@cuviper"],
90+
"compiler/rustc_middle/src/mir": ["compiler", "mir"],
91+
"compiler/rustc_middle/src/traits": ["compiler", "traits"],
92+
"compiler/rustc_mir/src/interpret": ["compiler", "mir"],
93+
"compiler/rustc_mir/src/transform": ["compiler", "mir-opt"],
94+
"compiler/rustc_mir_build/src/build": ["compiler", "mir"],
95+
"compiler/rustc_typeck": ["compiler", "typeck"],
96+
"compiler/rustc_traits": ["compiler", "traits"],
97+
"compiler/rustc_trait_selection": ["compiler", "traits"],
98+
"compiler/rustc_parse": ["compiler", "parser"],
99+
"compiler/rustc_parse/src/parse/lexer/": ["compiler", "lexer"],
100+
"compiler/rustc_query_impl": ["compiler", "query-system"],
101+
"compiler/rustc_query_system": ["compiler", "query-system"],
102+
"library/alloc": ["libs"],
103+
"library/core": ["libs", "@scottmcm"],
104+
"library/panic_abort": ["libs"],
105+
"library/panic_unwind": ["libs"],
106+
"library/proc_macro": ["@petrochenkov"],
107+
"library/std": ["libs"],
108+
"library/stdarch": ["libs"],
109+
"library/term": ["libs"],
110+
"library/test": ["libs"],
111+
"src/bootstrap": ["@Mark-Simulacrum"],
112+
"src/build_helper": ["@Mark-Simulacrum"],
113+
"src/ci": ["infra-ci"],
114+
"src/doc": ["doc"],
115+
"src/etc": ["@Mark-Simulacrum"],
116+
"src/librustdoc": ["rustdoc"],
117+
"src/llvm-project": ["@cuviper"],
118+
"src/stage0.txt": ["@Mark-Simulacrum"],
119+
"src/tools/cargo": ["@ehuss", "@joshtriplett"],
120+
"src/tools/compiletest": ["@Mark-Simulacrum"],
121+
"src/tools/linkchecker": ["@ehuss"],
122+
"src/tools/rust-installer": ["@Mark-Simulacrum"],
123+
"src/tools/rustbook": ["doc", "@ehuss"],
124+
"src/tools/rustdoc": ["rustdoc"],
125+
"src/tools/rustdoc-js": ["rustdoc"],
126+
"src/tools/rustdoc-js-std": ["rustdoc"],
127+
"src/tools/rustdoc-themes": ["rustdoc"],
128+
"src/tools/tidy": ["@Mark-Simulacrum"]
72129
},
73130
"mentions": {
74131
"src/rustdoc-json-types": {

‎highfive/tests/fakes.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,15 @@ def get_repo_configs():
4444
"some": ["all"],
4545
},
4646
},
47+
'nested_groups': {
48+
"groups": {
49+
"all": [],
50+
"a": ["@pnkfelix"],
51+
"b": ["@nrc"],
52+
"c": ["a", "b"]
53+
},
54+
"dirs": {"src/librustc_typeck": ["c"]},
55+
},
4756
'empty': {
4857
"groups": {"all": []},
4958
"dirs": {},

‎highfive/tests/test_newpr.py

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1160,6 +1160,17 @@ def test_circular_groups(self):
11601160
'rust', 'rust-lang', self.fakes['diff']['normal'], 'fooauthor'
11611161
)
11621162

1163+
def test_nested_groups(self):
1164+
"""Test choosing a reviewer from group with nested groups.
1165+
"""
1166+
self.handler = HighfiveHandlerMock(
1167+
Payload({}), repo_config=self.fakes['config']['nested_groups']
1168+
).handler
1169+
(chosen_reviewers, mentions) = self.choose_reviewers(
1170+
self.fakes['diff']['normal'], "nikomatsakis"
1171+
)
1172+
assert set(["pnkfelix", "nrc"]) == chosen_reviewers
1173+
11631174
def test_global_core(self):
11641175
"""Test choosing a reviewer from the core group in the global
11651176
configuration.

0 commit comments

Comments
 (0)