[GitHub] [struts] kusalk commented on a diff in pull request #728: Defines a proper CODEOWNERS file

2023-08-18 Thread via GitHub
kusalk commented on code in PR #728: URL: https://github.com/apache/struts/pull/728#discussion_r1298227020 ## CODEOWNERS: ## @@ -0,0 +1,2 @@ +# Request PR review from any Apache Struts committer +* @apache/struts-committers Review Comment: I see a list of 11 people - presuma

[GitHub] [struts] kusalk commented on a diff in pull request #728: Defines a proper CODEOWNERS file

2023-08-17 Thread via GitHub
kusalk commented on code in PR #728: URL: https://github.com/apache/struts/pull/728#discussion_r1296923283 ## CODEOWNERS: ## @@ -0,0 +1,2 @@ +# Request PR review from any Apache Struts committer +* @apache/struts-committers Review Comment: Ah it might be complaining because

[GitHub] [struts] kusalk commented on a diff in pull request #728: Defines a proper CODEOWNERS file

2023-08-17 Thread via GitHub
kusalk commented on code in PR #728: URL: https://github.com/apache/struts/pull/728#discussion_r1296910324 ## CODEOWNERS: ## @@ -0,0 +1,2 @@ +# Request PR review from any Apache Struts committer +* @apache/struts-committers Review Comment: I think GitHub doesn't like the `*`