Remove frequency defn - #21
Conversation
| Complain About: accidental-2119 yes, missing-example-ids yes | ||
| Assume Explicit For: yes | ||
| Die On: warning | ||
| WPT Path Prefix: cpu-performance |
There was a problem hiding this comment.
In this case we should just remove this line if we don't want to link to the WPTs. I think that should fix the need to have a hidden wpt element.
There was a problem hiding this comment.
I'm not sure what the common practice is regarding WPTs. We do have some (simple) tests in that file. I thought that including them as hidden (since we probably don't want to mention them in the text) was the correct way to link them to the spec. It did not cross my mind that we could leave them out entirely. If that's what you think is better, I'll change the PR.
There was a problem hiding this comment.
I think we should either unhide them or delete this line and not include them in the spec at all, and generally speaking I prefer (and most specs seem to do) the latter—deleting this line and making no mention of the existing WPTs in the spec.
2 participants
This PR removes the definition of CPU frequency, that caused the automated tests to fail.
Moreover, it converts double quotes to single quotes in JS code, for consistency.
Finally, it mentions the WPT tests that have been added.
It should fix issue #20.
Preview | Diff