Skip to content

Launch amp-script#23775

Merged
dreamofabear merged 3 commits intoampproject:masterfrom
dreamofabear:launch-amp-script
Aug 9, 2019
Merged

Launch amp-script#23775
dreamofabear merged 3 commits intoampproject:masterfrom
dreamofabear:launch-amp-script

Conversation

@dreamofabear
Copy link
Copy Markdown

Removes runtime experiment guards and "needs token" validation rule.

satisfies: "amp-experiment-token"
# Disabled because there are no active origin experiments; uncomment when
# there is at least one corresponding `requires` clause.
# satisfies: "amp-experiment-token"
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ampproject/wg-caching Validator changes FYI.

@dreamofabear dreamofabear requested a review from jridgewell August 7, 2019 20:05
@dreamofabear
Copy link
Copy Markdown
Author

/to @jridgewell

Copy link
Copy Markdown
Member

@twifkak twifkak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the heads up! Please add disabled_by: "transformed" per our past discussions around SXG, unless this has the expiry stuff we talked about and I missed it.

@twifkak
Copy link
Copy Markdown
Member

twifkak commented Aug 7, 2019

Oops, this is amp-script, not inline amp-script; ignore!

@dreamofabear dreamofabear merged commit 81b16d1 into ampproject:master Aug 9, 2019
@dreamofabear dreamofabear deleted the launch-amp-script branch August 9, 2019 21:04
Gregable pushed a commit that referenced this pull request Aug 16, 2019
* cl/262591581 Revision bump for #23840

* cl/262631527 Revision bump for #23765

* cl/263233041 data-ampdevmode. Avoid using non-data attributes on non-custom elements.

* cl/263556980 Allow nonce on `<link rel=stylesheet>`.

* cl/263618983 Revision bump for #23775

* cl/263623541 Revision bump for #23873

* cl/263628558 Revision bump for #23866

* cl/263637721 Revision bump for #23867

* cl/263656022 Revision bump for #23967

* fix reverse sync

* fix reverse sync
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants