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

Create qc-missing-deprecation-reason.sparql QC check #6969

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Create qc-missing-deprecation-reason.sparql QC check #6969

wants to merge 1 commit into from

Conversation

matentzn
Copy link
Member

@matentzn matentzn commented Dec 7, 2023 •
edited
Loading

@matentzn matentzn marked this pull request as draft December 7, 2023 20:43
@matentzn matentzn added the blocked Something needs to be done before work can proceed label Dec 7, 2023
@twhetzel
Copy link
Collaborator

twhetzel commented Dec 8, 2023

@matentzn is this "blocked" because a decision on #6968 is needed?

@matentzn
Copy link
Member Author

matentzn commented Dec 8, 2023

Yes! Blocked is a pretty useless tag as you never know when to check if it is "unblocked" but it helps when browsing open PRs to see that things need "unblocking" before proceeding

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sabrinatoro sabrinatoro Awaiting requested review from sabrinatoro

@twhetzel twhetzel Awaiting requested review from twhetzel twhetzel will be requested when the pull request is marked ready for review twhetzel is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@twhetzel twhetzel

Labels
blocked Something needs to be done before work can proceed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Do we want a QC check to ensure that all deprecated classes have an obsoletion reason?
2 participants
@matentzn @twhetzel