search

tdwg / tcs2

The TCS 2 Task Group will turn TCS into a form in which it can be maintained. The new version of TCS will be a vocabulary standard like Darwin Core and Audiovisual Core and will complement these other existing TDWG standards.
6 stars 0 forks source link

Remove traditionalSynonymyRelationshipType property #206

Closed nielsklazenga closed 2 years ago

nielsklazenga commented 2 years ago
nielsklazenga commented 2 years ago

It would have been better if I had not added the updated markdown/html to the PR. Will not do that anymore in following PRs.

nielsklazenga commented 2 years ago

@camwebb , this is not about whether we have the property or not – I'd rather not have it either – but about fixing errors in the text. The property is there with or without this PR, but at the moment it is required and has the notes of relationshipType. If only I could add and remove terms from the term list simply by doing a PR. I have copied your comment to issue #205, as I want to have the discussion too (but not here). I cannot remember why I did not just merge the PR but asked for a review. Probably because it was the first one I did and I felt that since I had asked for a PR I could not just go ahead and merge it.

nielsklazenga commented 2 years ago

@camwebb , I have now added a commit that does what I wanted to do in the first place.