Add base classes and fragment identifiers for vocabulary.

[msporny]

This PR attempts to address part of issue #1263 by defining base classes (and their anchors) in the VCDM specification.

---

💥 Error: 500 Internal Server Error 💥

PR Preview failed to build. (Last tried on Sep 26, 2023, 8:39 PM UTC).

More

PR Preview relies on a number of web services to run. There seems to be an issue with the following one:

🚨 Spec Generator - Spec Generator is the web service used to build specs that rely on ReSpec.

🔗 Related URL

😭  Sorry, there was an error generating the HTML. Please report this issue!
Specification: http://labs.w3.org/spec-generator/uploads/teRudZ/index.html?isPreview=true%3FisPreview%3Dtrue&publishDate=2023-09-26
ReSpec version: 34.1.8
File a bug: https://github.com/w3c/respec/
Error: undefined

If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.

[iherman]

Would it be possible to, in the same PR, make the changes on the vocabulary.yml file, too? For each of those classes a defined_by statement should replace the comment field.... This would save us yet another PR/merge round.

(I can make a PR against this one if you prefer.)

[iherman]

Would it be possible to, in the same PR, make the changes on the vocabulary.yml file, too? For each of those classes a defined_by statement should replace the comment field.... This would save us yet another PR/merge round.

(I can make a PR against this one if you prefer.)

Putting my money where my mouth is... I have created #1278 to do this.

@msporny, if you are o.k. with this, the best would be to merge #1278 right away into this one, so that all the possible comments would be at one place. I just did not want to directly edit your PR.

[iherman]

The issue was discussed in a meeting on 2023-09-14

• no resolutions were taken

View the transcript

1.6. Add base classes and fragment identifiers for vocabulary. (pr vc-data-model#1272)

See github pull request vc-data-model#1272.

Brent Zundel: primarily editorial, raised a few days ago.

Manu Sporny: Yes to all of that, just need to merge TallTed's changes.

See github pull request vc-data-model#1278.

Ivan Herman: For the record, I did 1278 yesterday.
� it's on top of 1272.

Manu Sporny: agree to merge on top of it.

Brent Zundel: merged, back to 1272.
� still have 3 more days for reviews.
� manu to look at suggestions made, anticipating that the PR will be merged soon.

Sebastian Crane: What I was saying is that the ease of merging Ivan's PR illustrates how useful it is to push to a 'personal' branch �within� the W3C repository, not to your GitHub fork. It is not possible to push to someone else's GitHub fork, so that discussion would have involved lots of git commands if Manu hadn't pushed to the main repository.

[msporny]

Normative, multiple reviews, changes requested and made, no objections, merging.