You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sling.apache.org by GitBox <gi...@apache.org> on 2021/06/29 11:13:55 UTC

[GitHub] [sling-org-apache-sling-jcr-repoinit] henrykuijpers opened a new pull request #20: SLING-10136 Add delete path operation

henrykuijpers opened a new pull request #20:
URL: https://github.com/apache/sling-org-apache-sling-jcr-repoinit/pull/20


   


-- 
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: dev-unsubscribe@sling.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [sling-org-apache-sling-jcr-repoinit] henrykuijpers commented on pull request #20: SLING-10136 Add delete path operation

Posted by GitBox <gi...@apache.org>.
henrykuijpers commented on pull request #20:
URL: https://github.com/apache/sling-org-apache-sling-jcr-repoinit/pull/20#issuecomment-882355086


   @bdelacretaz  Any thoughts?


-- 
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: dev-unsubscribe@sling.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [sling-org-apache-sling-jcr-repoinit] bdelacretaz commented on pull request #20: SLING-10136 Add delete path operation

Posted by GitBox <gi...@apache.org>.
bdelacretaz commented on pull request #20:
URL: https://github.com/apache/sling-org-apache-sling-jcr-repoinit/pull/20#issuecomment-894226396


   Sorry that I took so long to take a look at this.
   
   The code looks good to me.
   
   Same comment as in apache/sling-org-apache-sling-repoinit-parser#13 : in SLING-10136, Konrad expresses concerns about implementing this operation, I think those should be addressed before merging this.


-- 
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: dev-unsubscribe@sling.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [sling-org-apache-sling-jcr-repoinit] henrykuijpers commented on pull request #20: SLING-10136 Add delete path operation

Posted by GitBox <gi...@apache.org>.
henrykuijpers commented on pull request #20:
URL: https://github.com/apache/sling-org-apache-sling-jcr-repoinit/pull/20#issuecomment-882355086


   @bdelacretaz  Any thoughts?


-- 
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: dev-unsubscribe@sling.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org