Skip to content
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 more implementer details. #26

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Conversation

BigBlueHat
Copy link
Member

  • Add .editorconfig.
  • Fix whitespace in GrotoNetworking.json.
  • Add more metadata to implementations.

@aljones15
Copy link
Collaborator

can this be tested with a few test suites first? Properties in a manifest are often cast to the Endpoint class so adding new properties might result in some small correctable issues.

@BigBlueHat
Copy link
Member Author

@aljones15 yeah, there are also reporter updates to make. Mainly looking for format/shape feedback first.

This lays the foundation for eventually turning this metadata into VCs.

The context uses a mix of the W3Cs EARL vocabulary, DOAP (which EARL
uses), Schema.org, and a temporary `reporter` namespace mapped to
https://github.com/digitalbazaar/mocha-w3c-interop-reporter#
which is the primary software which consumes these details to generate
W3C test reports.

This context MAY (and likely should) move in the future either to that repo or
to some more permanent home.
Copy link

@davidlehn davidlehn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be good to find an EARL/DOAP expert if the idea is to properly align with those schemas. For instances, there is some mixing of schema.org and doap Person and Organization, which doesn't align with the DOAP schema ranges. In this use case, it probably doesn't matter. Not sure what best practices are with that.

Also the usage here isn't quite like what JSON-LD and rdf-canon use in their test results. Again, it might not matter here. This is headed the right direction in any case and will help improve reports.

"@type": "@id"
},
"vendor": {
"@id": "doap:vendor"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some of these probably should be @type: @id.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants