Skip to content

Conversation

@Thraka
Copy link
Contributor

@Thraka Thraka commented Nov 20, 2018

Label rules for dotnet/docs#9128

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Thraka! I think the rules need to be revisited to be repo specific. I commented where I saw we could change.

.ghal.rules.json Outdated
"defaults": [ ":watch: Not Triaged" ],
"product": {
"(?i).*": "Source - Docs.ms",
"(?i)dotnet-csharp$": ":books: Area - C# Guide",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think only the first one should be kept (Source - Docs.ms) - all the others don't make sense here

"(?i).*master\/docs\/standard\/modernize-with-azure-and-containers.*": ":card_file_box: Technology - .NET Architecture,:book: e-book - Modernizing w/ Windows containers",
"(?i).*master\/docs\/standard\/security.*": ":card_file_box: Technology - Security",
"(?i).*master\/docs\/standard\/serverless-architecture.*": ":card_file_box: Technology - .NET Architecture,:book: e-book - Serverless apps",
"(?i).*dotnet-api-docs\/blob\/master\/xml\/system\\.net.*": ":card_file_box: Technology - NCL",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keep just these for dotnet-api-docs, remove the rest

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mairaw Done

@Thraka
Copy link
Contributor Author

Thraka commented Nov 20, 2018

@mairaw can i just merge? or do i need to wait for build (which doesn't really apply here)

@mairaw
Copy link
Contributor

mairaw commented Nov 20, 2018

We tend to wait even when it doesn't apply. Up to you!

@Thraka Thraka merged commit f027aac into master Nov 20, 2018
@delete-merged-branch delete-merged-branch bot deleted the adegeo-ghalrules branch November 20, 2018 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants