-
Notifications
You must be signed in to change notification settings - Fork 17
add sovereignStates [4/7] #379
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
add sovereignStates [4/7] #379
Conversation
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.
"Rige" is correct spelling, should be added as an exception to the code spell
signybenediktsdottir
left a comment
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.
Could be merged but see comments on specific files first.
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.
NED is included in wikidata, abbreviation for Netherlands (https://en.wiktionary.org/wiki/NED). Should be added as an exception.
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.
Official name "Lebanon" also included as a synonym
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.
Country code "ISO 3166-1:LB" included in wikidata. Should this also be included on the instance?
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.
"Lubnan" could also be added as this is listed under native label on wikidata
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.
Official name "Lesotho" is included also as a synonym
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.
"Charu cha Malaŵi" is listed as native label on wikidata, could be added as a synonym
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.
Official name "Mali" also included as a synonym
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.
"bu" correct according to native label in wikidata for Wolof. Should be added as an exception to the code spell
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.
Official name "Moldova" is also listed as a synonym
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.
Offical name "Monaco" also listed as a synonym
|
@Raphael-Gazzotti for some countries the official name of the country is also included as a synonym. Could this be because the values from "native label" (wikidata) are included as a synonym? If that is the case, should I remove the official name listed under synonyms? |
|
@signybenediktsdottir Yes, we can remove them. They were included because I extracted them from other properties, but I should have removed them since they were already present for the property name. Sorry. |
|
Just removed the duplicates between the names and synonyms. The rest was already included, i.e., LB, Charu cha Malaŵi... |
changes to rerun checks (no changes were made)
d97df44
into
openMetadataInitiative:main
No description provided.