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/07/21 13:26:08 UTC

[GitHub] [sling-org-apache-sling-servlets-resolver] bdelacretaz commented on pull request #19: Sling 10644 return servlet resolver information in json format

bdelacretaz commented on pull request #19:
URL: https://github.com/apache/sling-org-apache-sling-servlets-resolver/pull/19#issuecomment-884188472


   > ...Compose json by hand instead of adding a new dependency to manage it...
   
   I think we need to define a nicer standard way of generating JSON in Sling, as we're doing that in many places.
   
   In this case I agree that not requiring a new dependency is good - no need to change this code, with the clean jsonpath tests I think it's appropriate. But I feel bad about us requiring people to use `printf` to generate JSON...


-- 
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