potiuk commented on PR #3966: URL: https://github.com/apache/jena/pull/3966#issuecomment-4746461422
Thanks `afs`, `rvesse` — applied your final batch: - Completed the Validations row (SHACL/ShEx imports fetch over HTTP → an SSRF surface). - Dropped the "(Draft one-liners — expand before publishing.)" placeholder in §11. - §11: removed the standalone `file:` from the SERVICE misuse line — per your note that `file:` in a query is a URI name, not a dereferenced local file (the model already carries that nuance in §6/§9, rvesse-confirmed). Title's already on "v1" with the repo names + the jena-text/validations split folded in, so I think that's the model done — ready to merge whenever you are. And yes, exactly `rvesse`: the first scan against this model is the real test — once results come back we iterate on §11a to suppress any false positives. Thanks both for the thorough review. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
