-
Notifications
You must be signed in to change notification settings - Fork 551
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.
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
fix: change the prefix for https://schema.org/
back to schema
#2312
Conversation
The default prefix for `https://schema.org/` registered with `rdflib.namespace.NamespaceManager` was inadvertently changed to `sdo` in 6.2.0, this however constitutes a breaking change, as code that was using the `schema` prefix would no longer have the same behavior. This change changes the prefix back to `schema`.
641ad7c
to
a65bfeb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
I really liked that you changed the default prefix back to sdo (no TLS) previous to RDFlib 6.3.2 ;-) Is there an easy way to override the default? I got this hint once but never tried it (lazy me). Background: We are still using (edited) http://schema.org (no TLS) in our SHACL shapes and Ontospy to document them. everything is fine in the RDF sense but schema1 simply looks odd in the docs. |
If you create a graph with You can see more details here - the |
@aucampia Great, thanks for the clarification! |
Summary of changes
The default prefix for
https://schema.org/
registered withrdflib.namespace.NamespaceManager
was inadvertently changed tosdo
in 6.2.0, this however constitutes a breaking change, as code that was using theschema
prefix would no longer have the same behavior. This change changes the prefix back toschema
.Checklist
the same change.
so maintainers can fix minor issues and keep your PR up to date.