-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Update and trim down scripts in /2dcontext/tools #7219
Conversation
Build PASSEDStarted: 2017-09-01 11:23:14 View more information about this build on: |
Remove --standalone option and mochitest business, but generate annotated-spec.html (not checked in), and fix various bugs. Update specextract.py and fetch a new copy of the spec. The "text" snippets in spec.yaml should be updated to match against the current spec, which can be worked on as followups.
68603d7
to
c9c39eb
Compare
Checks passed; ready for review. |
@@ -1,50 +1,74 @@ | |||
<?xml version="1.0" encoding="UTF-8"?><html xml:lang="en-US-x-hixie" xmlns="http://www.w3.org/1999/xhtml"><head><title>HTML Standard</title><link href="http://www.whatwg.org/style/specification" rel="stylesheet"/><link href="http://www.whatwg.org/images/icon" rel="icon"/><style> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I presume we can't just use a git submodule here rather than checking in a large part of the HTML spec?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dunno, I thought we generally try to avoid submodules? Also spec.yaml has text that's supposed to sync up with the spec, it seems clearer for reviewers if it's checked in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have plenty of other submodules used for such things; I thought it was required submodules that people didn't like. But yeah, I don't really care. shrugs
Remove --standalone option and mochitest business, but generate
annotated-spec.html (not checked in), and fix various bugs.
Update specextract.py and fetch a new copy of the spec.
The "text" snippets in spec.yaml should be updated to match against
the current spec, which can be worked on as followups.