Skip to content

Update xpath submodule to fix node-set string coercion (issue #182)#184

Merged
leonelsanchesdasilva merged 1 commit intomainfrom
issue-182
Mar 19, 2026
Merged

Update xpath submodule to fix node-set string coercion (issue #182)#184
leonelsanchesdasilva merged 1 commit intomainfrom
issue-182

Conversation

@leonelsanchesdasilva
Copy link
Copy Markdown
Collaborator

Points to the commit that fixes string functions (substring, contains, starts-with, etc.) incorrectly calling XNode.toString() instead of extracting text content when a node-set is passed as an argument.

Resolves #182.

Points to the commit that fixes string functions (substring, contains,
starts-with, etc.) incorrectly calling XNode.toString() instead of
extracting text content when a node-set is passed as an argument.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the xpath git submodule to a commit that fixes XPath string-function behavior when node-sets are passed (correct coercion to text content rather than XNode.toString()), resolving issue #182.

Changes:

  • Bumps the src/xpath/lib submodule pointer to include the node-set string coercion fix.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 78.4% 3351/4274
🟡 Branches 68.41% 1966/2874
🟡 Functions 71.9% 394/548
🟡 Lines 78.5% 3205/4083

Test suite run success

647 tests passing in 63 suites.

Report generated by 🧪jest coverage report action from 335c85e

@leonelsanchesdasilva leonelsanchesdasilva merged commit aa7aeea into main Mar 19, 2026
6 of 7 checks passed
@leonelsanchesdasilva leonelsanchesdasilva deleted the issue-182 branch March 19, 2026 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Functions in stylesheets are not evaluating correct values from the document

2 participants