Alling Readme to template #545
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
farshidtz
reviewed
May 15, 2024
| The latest Implementation Report is available [here](https://w3c.github.io/wot-discovery/testing/report.html). | ||
| The W3C WoT collects known implementations at <https://www.w3.org/WoT/developers/>. Implementations of Discovery are found under categories | ||
| "TD Directory (TDD) Implementations", "Runtime Implementations for TD Consumers" and "Runtime Implementations for TD Exposers". Further information | ||
| can also be found in the [implementations](implementations/README.md) folder. |
There was a problem hiding this comment.
These are pretty outdated. I suggest removing this file and maintaining the implementations in one place only.
|
|
||
| The latest Implementation Report is available [here](https://w3c.github.io/wot-discovery/testing/report.html). | ||
| The W3C WoT collects known implementations at <https://www.w3.org/WoT/developers/>. Implementations of Discovery are found under categories | ||
| "TD Directory (TDD) Implementations", "Runtime Implementations for TD Consumers" and "Runtime Implementations for TD Exposers". Further information |
There was a problem hiding this comment.
I'm not sure how useful it is to mention the heading titles. Is anyone going to read these here and try to find them there? Will anyone update the heading titles here if they change over time?
IMO, the developers page should have better grouping or labeling to make it clear which of those implement the discovery spec.
There was a problem hiding this comment.
I found that pretty weird too, but I was trying to be as close as possible to the template to avoid inconsistencies. We should mention this point in wot main repository and update the template.
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As discussed in the marketing and main call I've been assigned (see w3c/wot-marketing#461 (comment)) to upgrade the readme of this repo with the guidelines defined in the general template. This PR is a refactoring of the current README no information should be lost (please double check). Some warning points: