Skip to content

Update properties.json#88

Merged
apdavison merged 3 commits intomainfrom
lzehl_fixingVocabPropPlurals
Mar 12, 2026
Merged

Update properties.json#88
apdavison merged 3 commits intomainfrom
lzehl_fixingVocabPropPlurals

Conversation

@lzehl
Copy link
Copy Markdown
Member

@lzehl lzehl commented Mar 12, 2026

  • adding missing plurals
  • fixing wrong logic application (from myself yesterday) in some cases (@Raphael-Gazzotti must be plural in respect to the value of a property not in respect to the 'owner' of the property; e.g. isPartOf remains isPartOf or uses remains uses no matter if the value is plural or singular)

@apdavison can you please double check & merge if okay? (should solve also some conflicts we had yesterday with openMINDS_Python)

Note on the codespell error: This seems to fail because of the abbreviation TE which is not yet added to the exceptions of the codespell script.

@lzehl lzehl requested a review from apdavison March 12, 2026 06:50
@lzehl lzehl self-assigned this Mar 12, 2026
@lzehl lzehl added bug Something isn't working enhancement New feature or request labels Mar 12, 2026
Comment thread vocab/properties.json Outdated
Comment thread vocab/properties.json Outdated
@apdavison apdavison merged commit bd12ea8 into main Mar 12, 2026
1 of 2 checks passed
@Raphael-Gazzotti Raphael-Gazzotti deleted the lzehl_fixingVocabPropPlurals branch March 12, 2026 14:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants