-
Notifications
You must be signed in to change notification settings - Fork 75
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
Chore: Update missing Wikidata query forms #560
Conversation
Things that are still needed for the query generation process, @axif0:
The last thing is that we need the label service fields to not be removed by this process. Let's discuss later! 😊 |
We should check the cases when we're using the label service to return statements and maybe there are certain properties that we're returning for certain data types:
|
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.
Cool!
# tool: scribe-data | ||
# All arabic (Q13955) adverbs (Q380057) and their forms. | ||
# Enter this query at https://query.wikidata.org/. |
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.
nit - just a suggestion
Would it make sense to include a blurb in these headers that the files are being generated? With something along the lines of..
Auto-generated by the
check_and_update_missing_query_forms.yaml
workflow.
@@ -0,0 +1,21 @@ | |||
# tool: scribe-data |
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.
nit -
As the PR description links to the workflow, would it be a good idea to perhaps link to the specific commit version of the workflow that ran as opposed to simply the one found in main
?
i.e. instead of..
https://github.com/scribe-org/Scribe-Data/blob/main/.github/workflows/check_and_update_missing_query_forms.yaml
Mostly thinking really that this would allow someone to look at a closed automated PR and go to the correct version used - also this would allow for this as the workflow file gets renamed over time even.
I believe this could be done by passing in the parent commit SHA to pr_body.py that the workflow uses to create the PR description.
I updated the logic a bit for sub-language in forms. After update we get more missing forms as expected, also added the filter for sub-language you mentioned earlier.
Sub_lang for
English verb,
You can check the PR here. CC: @andrewtavis, @wkyoshida |
Nice, @axif0! Feel free to open a PR here and we'll dive into it more from there :) So exciting that this is being finalized! 🚀 |
Closing this as a new one will be opened up once #563 is merged :) |
Automated PR: Missing Lexeme Forms
This is an automated PR created by the Check and Update Missing Query Forms workflow.
Missing Forms Summary
Please review the changes and provide feedback.