kwin commented on code in PR #28:
URL: 
https://github.com/apache/sling-org-apache-sling-repoinit-parser/pull/28#discussion_r1053531692


##########
src/main/java/org/apache/sling/repoinit/parser/operations/SetAclPrincipalBased.java:
##########
@@ -27,7 +27,9 @@
 
 /** Set ACL statement that groups a set of AclLines
  *  that all refer to the same set of principals.
+ *  @deprecated Use {@link EnsureAclPrincipalBased} based with stricter 
semantics
  */
+@Deprecated
 @ProviderType
 public class SetAclPrincipalBased extends AclGroupBase {

Review Comment:
   Should we emit a WARN log message when the deprecated constructor is called 
to encourage users to migrate to the new statement?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to