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.

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

Replace all uses of DCE with DC and add violation check #692

Merged
merged 4 commits into from
Mar 1, 2023

Conversation

cthoyt
Copy link
Collaborator

@cthoyt cthoyt commented Feb 28, 2023

Closes #691

This PR does the following:

  1. Replaces all remaining uses of DCE with DC
  2. Removes the DCE prefix from the definitions and clean up corresponding declarations
  3. Adds a violation check to make sure no predicates use the URI prefix for DCE

When you run sh run.sh make sparql_test, there should be no errors.

@cthoyt cthoyt requested a review from balhoff February 28, 2023 21:20
Copy link
Contributor

@matentzn matentzn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are now getting to see how useful qc profile centralisation would be. @StroemPhi

@matentzn
Copy link
Contributor

matentzn commented Mar 1, 2023

Ready to merge after conflict resolution

@cthoyt
Copy link
Collaborator Author

cthoyt commented Mar 1, 2023

@matentzn merge/conflict resolution is done. Next steps:

  1. Upstream these QC profiles into ODK
  2. How do we ensure that sparql_test is run as part of QC for pull requests?

@matentzn
Copy link
Contributor

matentzn commented Mar 1, 2023

Upstream these QC profiles into ODK

PR here: https://github.com/INCATools/ontology-development-kit/tree/master/template/src/sparql

How do we ensure that sparql_test is run as part of QC for pull requests?

It always is:

https://github.com/oborel/obo-relations/pull/692/checks#step:4:26

@matentzn matentzn merged commit ab5b277 into oborel:master Mar 1, 2023
@cthoyt cthoyt deleted the dce-gone branch March 1, 2023 13:43
cthoyt added a commit to cthoyt/ontology-development-kit that referenced this pull request Mar 1, 2023
anitacaron added a commit to INCATools/ontology-development-kit that referenced this pull request Oct 4, 2023
* Add SPARQL to find uses of deprecated DC

This is a follow-up to oborel/obo-relations#692

* remove misplaced sparql query

* add `dc-properties` custom sparql check

* update file name

* filter on terms using ontology namespaces

---------

Co-authored-by: Anita Caron <anitacaron@users.noreply.github.com>
Co-authored-by: Anita Caron <anitac@ebi.ac.uk>
Co-authored-by: Nico Matentzoglu <nicolas.matentzoglu@gmail.com>
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.

Deprecate use of DC elements namespace
3 participants