Skip to content

Fix QNAME grammar to match Turtle/SPARQL PN_LOCAL#136

Merged
MaillPierre merged 2 commits into
developfrom
bugfix/133-fix-qname-grammar
Apr 24, 2025
Merged

Fix QNAME grammar to match Turtle/SPARQL PN_LOCAL#136
MaillPierre merged 2 commits into
developfrom
bugfix/133-fix-qname-grammar

Conversation

@remiceres
Copy link
Copy Markdown
Contributor

Changelog\n- Accepts '.', '_' and escaped chars in local names.

* Adds helper tokens ESC, PERCENT, PLX.
* Updates QNAME lexical rule to conform to Turtle/SPARQL.
* Moves helper tokens inside TOKEN block to fix JavaCC error (#XYZ).
@remiceres remiceres requested a review from MaillPierre April 23, 2025 09:55
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 23, 2025

Test Results

358 tests  ±0   358 ✅ ±0   24s ⏱️ -2s
 12 suites ±0     0 💤 ±0 
 12 files   ±0     0 ❌ ±0 

Results for commit 5dd2fd7. ± Comparison against base commit 7cafa4b.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

Overall Project 46.34% 🍏
Files changed 70.73% 🍏

File Coverage
SparqlCoreseTokenManager.java 55.54% -0.07% 🍏

@MaillPierre MaillPierre merged commit 90f940e into develop Apr 24, 2025
2 checks passed
@remiceres remiceres deleted the bugfix/133-fix-qname-grammar branch April 24, 2025 14:43
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.

2 participants