Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: remove rdf-js stub #634

Closed
wants to merge 55 commits into from
Closed

fix: remove rdf-js stub #634

wants to merge 55 commits into from

Conversation

jeswr
Copy link
Contributor

@jeswr jeswr commented Feb 6, 2024

As discussed in hypermedia-app/hydra-validator#122 this stub should not be in-use anywhere

@bourgeoa
Copy link
Member

bourgeoa commented Feb 7, 2024 •
edited
Loading

Thanks. can you rebase to fix/remove-rdfjs-stub
Tests do fail.

@jeswr jeswr changed the base branch from main to fix/remove-rdfjs-stub February 7, 2024 21:10
@jeswr
Copy link
Contributor Author

jeswr commented Feb 7, 2024

Thanks for the review @bourgeoa - all comments should now be addressed

jeswr and others added 25 commits February 7, 2024 21:12
@jeswr
chore: add node 20 to CI
e262aaa 
Saw this in passing, node 20 is now LTS
@bourgeoa
remove only document content
e30ad52
@bourgeoa
add commment
9ca72b5
@bourgeoa
Merge pull request linkeddata#633 from linkeddata/issue#631
7a40abf 
removeMetadata
@bourgeoa
Revert "removeMetadata"
cc97e88
@bourgeoa
Merge pull request linkeddata#641 from linkeddata/revert-633-issue#631
ee0ceb0 
Revert "removeMetadata"
@bourgeoa
removeMetedata()
f3ac957
@bourgeoa
test response exists
8b6d35b
@bourgeoa
comment
ee32c0f
@bourgeoa
Merge pull request linkeddata#642 from linkeddata/fix/issue#630
2e8d292 
test response exists
@bourgeoa
null/undefined
ae142f3
@bourgeoa
Merge pull request linkeddata#638 from linkeddata/fix/issue#637
2d89176 
only remove document content
@bourgeoa
Merge pull request linkeddata#643 from linkeddata/fix/issue#639
3e2aa53 
null/undefined
@bourgeoa
keep status lookup
694a1c2
@bourgeoa
Merge pull request linkeddata#644 from linkeddata/removeMetadata
e72fe6d 
Remove metadata
@bourgeoa
2.2.34-beta
cd9a596
@bourgeoa
start remove status collection
f821623
@bourgeoa
remove status first
bf0b7e1
@bourgeoa
2.2.34-beta3
dad4052
@bourgeoa
replace removeMatches
c45ae25
@bourgeoa
Merge pull request linkeddata#645 from linkeddata/rmMetadata
e2aa46e 
remove metadata
@bourgeoa
2.2.34
de0aab4
@bourgeoa
update jsonld
25d13ff
@bourgeoa
update minor dependencies
14c742e
@bourgeoa
[email protected]
06ce626
bourgeoa and others added 27 commits April 21, 2024 18:34
@bourgeoa
Merge pull request linkeddata#632 from linkeddata/issue#75
b56cd73 
XML Literal in browser
@bourgeoa
2.2.35
0aa77d2
@bourgeoa
Merge branch 'main' of https://github.com/linkeddata/rdflib.js into main
43709c6
@pduchesne @angelo-v
fix number regex to accept upper and lowercase E notation
cfc0f8c
@RinkeHoekstra
Passing this.flags to stringToN3 to ensure that format flags are prop…
0d9e8a8 
…erly used. Fixes linkeddata#650
@Kibubu @angelo-v
Fix quotation marks inside code
69fad70 
The code examples contained a mix of slanted, opening and closing quotations marks and could not be copy pasted into your playground.

Example: $rdf.sym(‘’https://example.com/alice/card”); 

Fixed by replacing all marks with either " or ' and fixed unmatched pairs.
@Kibubu @angelo-v
Fix typo in example code
82f693d
@bourgeoa
move @babel/runtime to dev
12b586c
@bourgeoa
Update ci.yml
0f0440d 
actions artifacts v4
@bourgeoa
update caniuse-lite
4677215
@bourgeoa
Update ci.yml
b81eb5d 
remove : - rm-r .gitignore
@bourgeoa
Merge pull request linkeddata#652 from RinkeHoekstra/ISSUE-650
fde7a68 
Pass this.flags to `stringToN3`, fixes linkeddata#650.
@bourgeoa
Merge pull request linkeddata#635 from jeswr/patch-1
2bb9ed0 
chore: add node 20 to CI
@bourgeoa
priorize SPARQL patch
d00b4e2
@bourgeoa
Merge pull request linkeddata#662 from linkeddata/priorizeSPARQL
7167422 
priorize SPARQL patch
@bourgeoa
Update ci.yml
cb2ecad
@bourgeoa
2.2.36
2888e04
@bourgeoa
update solid-namespace
806b043
@bourgeoa
minor dependencies
bb406f0
@bourgeoa
minor dependencies
c19fb8d
@bourgeoa
minor dependencies
81fd5a7
@bourgeoa
Merge pull request linkeddata#664 from linkeddata/eslint@9
9582a32 
minor dependencies
@bourgeoa
2.2.37
b210645
@jeswr
fix: remove rdfjs stub
1e9674b
@jeswr
chore: use rdfjs/types in tests and fix compatibility issues
3c6e286
@jeswr
chore: make type defs consistent
b5d3bff
@jeswr
chore: gather updates from main
a98a591
@jeswr jeswr force-pushed the fix/remove-rdfjs-stub branch from 44a10d4 to a98a591 Compare March 8, 2025 11:59
@jeswr jeswr closed this Mar 8, 2025
@jeswr jeswr mentioned this pull request Mar 8, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@jeswr @bourgeoa @pduchesne @RinkeHoekstra @Kibubu