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

Do not validate for #development=1 when on AMP Caches #19096

Merged
merged 6 commits into from Nov 2, 2018
Merged

Do not validate for #development=1 when on AMP Caches #19096

merged 6 commits into from Nov 2, 2018

Conversation

honeybadgerdontcare
Copy link
Contributor

Fixes #19082

@honeybadgerdontcare honeybadgerdontcare merged commit 67b5c0b into ampproject:master Nov 2, 2018
@honeybadgerdontcare honeybadgerdontcare deleted the amp-cache branch November 2, 2018 22:19
twifkak added a commit to twifkak/amphtml that referenced this pull request Nov 2, 2018
@twifkak twifkak mentioned this pull request Nov 2, 2018
twifkak added a commit that referenced this pull request Nov 3, 2018
* cl/219531121 Revision bump for #17481

* cl/219850294 Allow `http` scheme for links in email spec

* cl/219866890 Revision bump for #19043

* cl/219867113 Revision bump for #18981

* cl/219877087 Make ESlint happy.

* cl/219882876 Fix lint for #19096
Enriqe pushed a commit to Enriqe/amphtml that referenced this pull request Nov 28, 2018
* do not validate AMP Cache URLs with #development=1

* add cloudflare

* clean up

* eslint fixes

* whitelist validator-in-browser

* whitelist
Enriqe pushed a commit to Enriqe/amphtml that referenced this pull request Nov 28, 2018
* cl/219531121 Revision bump for ampproject#17481

* cl/219850294 Allow `http` scheme for links in email spec

* cl/219866890 Revision bump for ampproject#19043

* cl/219867113 Revision bump for ampproject#18981

* cl/219877087 Make ESlint happy.

* cl/219882876 Fix lint for ampproject#19096
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

4 participants