Skip to content

Commit be61f62

Browse files
committed
Roles: Move general in same section as sub-package
maintainers Wording tweaks Consistent verbiage
1 parent 74556f1 commit be61f62

File tree

1 file changed

+34
-31
lines changed

1 file changed

+34
-31
lines changed

roles.json

Lines changed: 34 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -336,30 +336,18 @@
336336
}
337337
},
338338
{
339-
"role": "Core astropy package maintainer (general)",
340-
"url": "Core_package_general_maintainer",
341-
"people": [
342-
"Derek Homeier",
343-
"Pey Lian Lim",
344-
"Ole Streicher"
345-
],
346-
"role-head": "Core astropy package maintainer (general)",
347-
"responsibilities": {
348-
"description": "Maintain the astropy core package in aspects that are not specific to a single sub-package",
349-
"details": [
350-
"Evaluating new pull requests for quality, API consistency, Astropy coding standards, and appropriateness within the overall astropy ecosystem, in particular for pull requests spanning multiple sub-packages",
351-
"Merging Pull Requests that are non-controversial or after reaching out to relevant subpackage maintainers",
352-
"Maintain, review, and advocate for useful interaction between multiple sub-packages",
353-
"Perform initial triaging of issues and pull requests",
354-
"Keeping track of frequent contributors and their relevant areas of expertise"
355-
]
356-
}
357-
},
358-
{
359-
"role": "Core astropy package maintainer (sub-package)",
339+
"role": "Core astropy package maintainer",
360340
"url": "Subpackage_maintainer",
361341
"role-head": "Sub-package maintainer (at least one per core package sub-package)",
362342
"sub-roles": [
343+
{
344+
"role": "General",
345+
"people": [
346+
"Derek Homeier",
347+
"Pey Lian Lim",
348+
"Ole Streicher"
349+
]
350+
},
363351
{
364352
"role": "astropy.constants",
365353
"people": [
@@ -503,16 +491,31 @@
503491
]
504492
}
505493
],
506-
"responsibilities": {
507-
"description": "Maintain a sub-package of the astropy core package, including:",
508-
"details": [
509-
"Evaluating new pull requests for quality, API consistency, Astropy coding standards, and appropriateness within the overall astropy ecosystem",
510-
"Merging Pull Requests in the sub-package",
511-
"Keeping track of the \u201cbig picture\u201d progress of the sub-package - includes new feature development and significant bugs",
512-
"Perform initial triaging of issues and pull requests",
513-
"Keeping track of frequent contributors to the sub-package and their relevant areas of expertise"
514-
]
515-
}
494+
"responsibilities": [
495+
{
496+
"subrole-head": "General maintainer",
497+
"description": "Maintain the astropy core package in aspects that are not specific to a single sub-package, including:",
498+
"details": [
499+
"Evaluating new pull requests for quality, API consistency, Astropy coding standards, and appropriateness within the overall astropy ecosystem, in particular for pull requests spanning multiple sub-packages",
500+
"Merging Pull Requests that are non-controversial or after reaching out to relevant subpackage maintainers",
501+
"Maintain, review, and advocate for useful interaction between multiple sub-packages",
502+
"Perform initial triaging of issues and pull requests",
503+
"Keeping track of frequent contributors and their relevant areas of expertise"
504+
]
505+
},
506+
{
507+
508+
"subrole-head": "Sub-package maintainer",
509+
"description": "Maintain a sub-package of the astropy core package, including:",
510+
"details": [
511+
"Evaluating new pull requests for quality, API consistency, Astropy coding standards, and appropriateness within the overall astropy ecosystem",
512+
"Merging Pull Requests in the sub-package",
513+
"Keeping track of the \u201cbig picture\u201d progress of the sub-package - includes new feature development and significant bugs",
514+
"Perform initial triaging of issues and pull requests",
515+
"Keeping track of frequent contributors to the sub-package and their relevant areas of expertise"
516+
]
517+
}
518+
]
516519
},
517520
{
518521
"role": "Coordinated package maintainer",

0 commit comments

Comments
 (0)