Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Add upper version limit for rdflib #131

Merged
merged 2 commits into from
Apr 21, 2020
Merged

Add upper version limit for rdflib #131

merged 2 commits into from
Apr 21, 2020

Conversation

MasterOdin
Copy link
Collaborator

rdflib 5.0.0 reworked some amount of their interface, which breaks things here. Minimally, SPARQLWrapper is no longer used, and a requests based wrapper internal to rdflib is now used, which breaks compatibility with older RDF stores that do not follow the SPARQL Protocol 1.1 spec. Setting this to minimize the break here, with a look to move to rdflib 5 in the next major version bump of gutenberg itself.

rdflib 5.0.0 reworked some amount of their interface, which breaks things here. Minimally, SPARQLWrapper is no longer used,
and a requests based wrapper internal to rdflib is now used, which breaks compatibility with older RDF stores that do not follow
the SPARQL Protocol 1.1 spec. Setting this to minimize the break here, with a look to move to rdflib 5 in the next major version
bump of gutenberg itself.

Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
@codecov-io
Copy link

codecov-io commented Apr 19, 2020

Codecov Report

Merging #131 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #131   +/-   ##
=======================================
  Coverage   95.74%   95.74%           
=======================================
  Files          29       29           
  Lines        1033     1033           
=======================================
  Hits          989      989           
  Misses         44       44           
Impacted Files Coverage Δ
setup.py 0.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 378a7b4...3a7fca9. Read the comment docs.

@MasterOdin
Copy link
Collaborator Author

Will fix the azure pipelines and then merge.

@MasterOdin MasterOdin force-pushed the MasterOdin-patch-2 branch 2 times, most recently from f82a45e to 7c02389 Compare April 21, 2020 01:30
Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
@MasterOdin MasterOdin merged commit 2004353 into master Apr 21, 2020
@c-w c-w deleted the MasterOdin-patch-2 branch August 3, 2020 13:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants