Skip to content

Commit 99831c2

Browse files
committed
Roles: Move general in same section as sub-package
maintainers Wording tweaks Consistent verbiage
1 parent edfc752 commit 99831c2

File tree

1 file changed

+35
-32
lines changed

1 file changed

+35
-32
lines changed

roles.json

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

0 commit comments

Comments
 (0)