azure-sdk-for-net icon indicating copy to clipboard operation
azure-sdk-for-net copied to clipboard

Include NetworkCloud to the CodeOwner file

Open cn0238 opened this issue 1 year ago • 1 comments

updating codeowner file

cn0238 avatar Aug 27 '24 23:08 cn0238

Thank you for your contribution @cn0238! We will review the pull request and get back to you soon.

github-actions[bot] avatar Aug 27 '24 23:08 github-actions[bot]

Added reformatted label to match repository naming pattern for network subcategories. Will need to kick the linter again tomorrow after it refreshes the label list.

jsquire avatar Aug 29 '24 21:08 jsquire

@JimSuplizio: The label that is being flagged by the linter exists in the repository with exact capitalization and spacing. Any idea why the linter is rejecting it?

jsquire avatar Aug 30 '24 15:08 jsquire

We are working to get the NetworkCloud PR Label registered.

cn0238 avatar Aug 30 '24 16:08 cn0238

/azp run net - codeowners-linter - ci

JimSuplizio avatar Aug 30 '24 19:08 JimSuplizio

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Aug 30 '24 19:08 azure-pipelines[bot]

@JimSuplizio: The label that is being flagged by the linter exists in the repository with exact capitalization and spacing. Any idea why the linter is rejecting it?

@jsquire if you looked at the date on the failed run it was from yesterday. I reran it today and it passed. If the label was added yesterday it wouldn't be in the repo/label cache until this morning and that would explain this.

JimSuplizio avatar Aug 30 '24 19:08 JimSuplizio

We are working to get the NetworkCloud PR Label registered.

This will be Network - Cloud and has been created in the .NET repository. There's a number of Network - labels, and we'll want this to follow that pattern.

//cc: @ronniegeraghty

jsquire avatar Aug 30 '24 21:08 jsquire

@JimSuplizio: The label that is being flagged by the linter exists in the repository with exact capitalization and spacing. Any idea why the linter is rejecting it?

@jsquire if you looked at the date on the failed run it was from yesterday. I reran it today and it passed. If the label was added yesterday it wouldn't be in the repo/label cache until this morning and that would explain this.

Dang it. Sorry. I swore that I kicked that over again this morning.

jsquire avatar Aug 30 '24 21:08 jsquire

No, we don't want to be incorporated under Network - Network Cloud as a subcategory, but wish to be NetworkCloud.

cn0238 avatar Aug 30 '24 21:08 cn0238

@cn0238: Repository labels are chosen by the Azure SDK team and generally are required to follow our patterns. @ronniegeraghty has final approval on label names and will request any adjustments based on branding concerns and likewise.

jsquire avatar Aug 30 '24 22:08 jsquire