FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Make _is_valid_absolute_iri also count hash chars. by mielvds · Pull Request #316 · digitalbazaar/pyld · GitHub

/ pyld Public

Make _is_valid_absolute_iri also count hash chars. - #316

Merged
mielvds merged 5 commits into
v4.0.0from
fix-toRdf-te111-te112
Aug 24, 2026
ConversationCommits (5)ChecksFiles changed
Merged

Make _is_valid_absolute_iri also count hash chars.#316
mielvds merged 5 commits into
v4.0.0from
fix-toRdf-te111-te112

Conversation

mielvds commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

RDF IRI validation in so IRIs with multiple # fragments are skipped during toRDF.

Fixes toRdf#te111 and toRdf#te112.

github-actions Bot commented Aug 14, 2026 •
edited
Loading

Copy link
Copy Markdown

Coverage Report
FileStmtsMissCoverMissing
lib/pyld
   canon.py2791595%35, 46, 49–54, 78–79, 508, 530, 594, 604, 658–659
   context_resolver.py103892%78, 137, 151, 197, 203–205, 229
   iri_resolver.py141398%311–312, 322
   jsonld.py249517493%244–248, 297, 334, 339, 349, 366–381, 417–418, 460, 468, 488, 496–497, 605–610, 690–691, 706–707, 767–772, 797–798, 813–814, 824–825, 851, 897, 906–912, 928–929, 971, 978, 985, 1050, 1075, 1097–1099, 1109–1112, 1127, 1150, 1162–1165, 1261, 1270–1273, 1293, 1322, 1355, 1359, 1392–1397, 1410–1412, 1422–1424, 1467, 1486–1489, 1502, 1551, 1654–1661, 1688–1690, 1716–1719, 1895, 1933, 2239, 2370, 2379–2386, 2480, 2507, 2531, 2539, 3133, 3183, 3202, 3267, 3320, 3323, 3386, 3471, 3505, 3513, 3791, 3796, 3798, 3993–3995, 4144, 4207, 4446, 4734, 4866, 4900, 4929–4932, 4954–4955, 5138, 5202, 5229, 5303, 5305, 5323, 5421, 5470, 5702, 5704, 5743, 5745, 5754, 5896, 6024, 6111, 6231–6245, 6307, 6309, 6377, 6604, 6610–6614, 6653, 6656, 6746–6747, 6756
   util.py72396%43, 116, 136
lib/pyld/documentloader
   aiohttp.py712269%29–39, 68, 74, 86, 98–106, 118–121, 138, 153–156
   file.py36197%49
   requests.py52590%57, 71, 83–91
   scheme_directed.py15193%38
TOTAL347423293% 

Tests Skipped Failures Errors Time
1842 29 💤 0 ❌ 0 🔥 3m 6s ⏱️

mielvds force-pushed the fix-toRdf-te111-te112 branch from 4b73959 to 5e98de0 Compare August 20, 2026 12:17
Base automatically changed from fix-toRdf-twf05 to v4.0.0 August 24, 2026 08:54
mielvds force-pushed the fix-toRdf-te111-te112 branch from 045297b to c3dda4d Compare August 24, 2026 08:55
mielvds merged commit 34e47d9 into v4.0.0 Aug 24, 2026
19 checks passed
mielvds deleted the fix-toRdf-te111-te112 branch August 27, 2026 14:00
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

anatoly-scherbakov anatoly-scherbakov approved these changes
BigBlueHat Awaiting requested review from BigBlueHat
davidlehn Awaiting requested review from davidlehn

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

v4.0.0 (migration to rdflib; toRdf co...

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL