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

Validator Rollup #6284

Merged
merged 6 commits into from Nov 21, 2016
Merged

Validator Rollup #6284

merged 6 commits into from Nov 21, 2016

Conversation

Gregable
Copy link
Member

No description provided.

@Gregable Gregable merged commit d9d5cb4 into ampproject:master Nov 21, 2016
@Gregable Gregable deleted the rollup branch November 21, 2016 23:29
Lith pushed a commit to Lith/amphtml that referenced this pull request Dec 22, 2016
* revision bump due to amp-hulu extension

* Support both v0.js and amp4ads-v0.js if the html_format is AMP4ADS

* Test in javascript that validator rules should make sense.

* Implement new validator logic that disallows the `<base href>` tag from appearing after any URL on the page.

* Test ValidationError in javascript.

* Allow this usage of startsWith to pass travis.
Lith pushed a commit to Lith/amphtml that referenced this pull request Dec 22, 2016
* revision bump due to amp-hulu extension

* Support both v0.js and amp4ads-v0.js if the html_format is AMP4ADS

* Test in javascript that validator rules should make sense.

* Implement new validator logic that disallows the `<base href>` tag from appearing after any URL on the page.

* Test ValidationError in javascript.

* Allow this usage of startsWith to pass travis.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants