Skip to content
This repository has been archived by the owner on Aug 26, 2022. It is now read-only.

Meta: enable HTML checker #44

Merged
merged 1 commit into from
Apr 9, 2017
Merged

Meta: enable HTML checker #44

merged 1 commit into from
Apr 9, 2017

Conversation

annevk
Copy link
Member

@annevk annevk commented Jan 13, 2017

No description provided.

@annevk
Copy link
Member Author

annevk commented Jan 13, 2017

@sideshowbarker I guess we need to decide whether those resources you checked in need to be preserved and excluded from validation or whether we need to fix them. It seems this has found a bunch of legitimate errors too though that are worth fixing.

@annevk
Copy link
Member Author

annevk commented Jan 13, 2017

(I'm happy to fix all errors, but want your opinion first.)

@sideshowbarker
Copy link
Member

I think we should skip checking of everything in the webapps and workers subdirectories, but fix any errors in the root index.html file and https://github.com/whatwg/platform.html5.org/blob/master/specs/index.html and https://github.com/whatwg/platform.html5.org/blob/master/history/index.html

@annevk
Copy link
Member Author

annevk commented Jan 13, 2017

@sideshowbarker maybe you can provide the fix for .travis.yml that makes it skip those directories?

@sideshowbarker
Copy link
Member

@sideshowbarker maybe you can provide the fix for .travis.yml that makes it skip those directories?

yes (assigning to myself)

@sideshowbarker sideshowbarker self-assigned this Jan 13, 2017
@annevk
Copy link
Member Author

annevk commented Mar 20, 2017

@sideshowbarker ping.

@sideshowbarker
Copy link
Member

Will get back to this asap (this weekend)

@sideshowbarker sideshowbarker merged commit dad8149 into master Apr 9, 2017
@sideshowbarker sideshowbarker deleted the annevk/checker branch April 9, 2017 04:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants