[GitHub] [sling-org-apache-sling-servlets-resolver] cziegeler commented on pull request #31: SLING-11558 part1: remove the LocationIterator

["cziegeler (via GitHub)" <gi...@apache.org>]

cziegeler commented on PR #31:
URL: https://github.com/apache/sling-org-apache-sling-servlets-resolver/pull/31#issuecomment-1524996733

   By looking at the code changes, it is impossible for me to decide whether there is a breaking change lurking in some edge case. This is at the heart of Sling and I would avoid touching it.
   That said, *if* you want to go ahead with it, please make a release of the bundle with the *current* state first, before applying any PR. This way this change is not interfering with other changes currently not released.


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