Update CODEOWNERS #1995
Update CODEOWNERS #1995
Conversation
| @@ -1,3 +1,3 @@ | |||
| #ECCN: Open Source | |||
| #GUSINFO: Education Data Architecture (EDA), Education Data Architecture (EDA) | |||
|
|
|||
| #GUSINFO: Rad-ish, Education Data Architecture & K-12 (MP) | |||
There was a problem hiding this comment.
Are spaces okay? The other codeowners PRs don't have spaces, so just want to make sure this is fine
There was a problem hiding this comment.
It's a good question - I was going off the confluence page where the examples show spaces in the names of product tag, at least.
https://confluence.internal.salesforce.com/display/corescm/Create+a+GUS-Aware+CODEOWNERS+File
There was a problem hiding this comment.
Okay. I just didn't know if "#GUSINFO: Rad-ish, Education Data Architecture & K-12 (MP)" vs "#GUSINFO:Rad-ish,Education Data Architecture & K-12 (MP)" made a difference, as I noticed all of the others are the latter.
There was a problem hiding this comment.
It seems they trim the line when the split the string (based on #GUSINFO) so I believe it should be ok with a space.
|
Closing + opened a new PR with correct branch naming |
No description provided.