Update CODEOWNERS with new assignments and corrections#21894
Update CODEOWNERS with new assignments and corrections#21894Generalcamo wants to merge 3 commits intomasterfrom
Conversation
Signed-off-by: Cody Brittain <1779662+Generalcamo@users.noreply.github.com>
|
!review |
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports
|
Signed-off-by: Cody Brittain <1779662+Generalcamo@users.noreply.github.com>
FabianK3
left a comment
FabianK3
left a comment
There was a problem hiding this comment.
Persistence is only going to grow in the future, I don't think it could hurt to add unless somebody has objections on it.
Signed-off-by: Cody Brittain <1779662+Generalcamo@users.noreply.github.com>
FabianK3
left a comment
FabianK3
left a comment
There was a problem hiding this comment.
I would consolidate the persistent lines as the comment describes, based of Matts response, otherwise everything LGTM.
The kind of PR that requires at least a maintainer to look over it for obvious reasons...
| # Persistence, Fabian's area | ||
| **/persistence @NonQueueingMatt @FabianK3 | ||
| /code/__DEFINES/persistence.dm @NonQueueingMatt @FabianK3 | ||
| /code/__HELPERS/logging/subsystems/persistence.dm @Arrow768 @NonQueueingMatt @FabianK3 | ||
| /code/controllers/subsystems/persistence.dm @NonQueueingMatt @FabianK3 |
There was a problem hiding this comment.
I would suggest that for every kind it's me + Arrow and optionally if @NonQueueingMatt wants that as well.
I guess Arrow + Matt is always the default fallback if in doubt.
2 participants
Removing old team members, adding new ones in their place.
NUFC