rvesse opened a new pull request, #1640:
URL: https://github.com/apache/jena/pull/1640
Expands the path flattening test coverage so that all tests cover both the
default and the algebra based transforms. Also makes some additional fixes to
both transforms for parity in a couple of areas:
- Sequence transforms place the grounded portion of the path first in the
sequence
- Default transform now explicitly errors on invalid `:p{N,M}` forms (e.g.
`:p{4,2}`) instead of generating the wrong algebra
Provides more testing around the path transformation aspect of #1629
----
- [x] Tests are included.
- [x] Commits have been squashed to remove intermediate development commit
messages.
By submitting this pull request, I acknowledge that I am making a
contribution to the Apache Software Foundation under the terms and conditions
of the [Contributor's
Agreement](https://www.apache.org/licenses/contributor-agreements.html).
----
See the [Apache Jena "Contributing"
guide](https://github.com/apache/jena/blob/main/CONTRIBUTING.md).
--
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]
