Skip to content

Validator rollup#29698

Merged
twifkak merged 8 commits intoampproject:masterfrom
twifkak:rollup
Aug 5, 2020
Merged

Validator rollup#29698
twifkak merged 8 commits intoampproject:masterfrom
twifkak:rollup

Conversation

@twifkak
Copy link
Copy Markdown
Member

@twifkak twifkak commented Aug 5, 2020

@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Aug 5, 2020

Hey @gmajoulet, @newmuis, @Enriqe! These files were changed:

extensions/amp-story/validator-amp-story.protoascii

Hey @ampproject/wg-caching! These files were changed:

validator/engine/htmlparser-interface.js
validator/engine/htmlparser.js
validator/engine/validator.js
validator/engine/validator_test.js
validator/nodejs/index.js
validator/testdata/actions_feature_tests/amp_bind.html
validator/testdata/actions_feature_tests/amp_bind.out
validator/testdata/actions_feature_tests/amp_form.html
validator/testdata/actions_feature_tests/amp_form.out
validator/testdata/actions_feature_tests/amp_list.html
validator/testdata/actions_feature_tests/amp_list.out
validator/testdata/actions_feature_tests/css_length.html
+23 more

@google-cla google-cla bot added the cla: yes label Aug 5, 2020
@honeybadgerdontcare
Copy link
Copy Markdown
Contributor

@GeorgeLuo FYI some validator changes: (1) AMP Email has a new document size byte limit, currently 100k and (2) AMP Actions is deprecated and removed from the Validator.

Copy link
Copy Markdown
Contributor

@Enriqe Enriqe left a comment

Choose a reason for hiding this comment

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

Approval for amp-story.protoascii

This should have been part of e498e1e.
@twifkak twifkak merged commit d53c7be into ampproject:master Aug 5, 2020
@twifkak twifkak deleted the rollup branch August 5, 2020 22:51
ed-bird pushed a commit to ed-bird/amphtml that referenced this pull request Dec 10, 2020
* cl/323700659 Revision bump for ampproject#29329
* cl/323797874 Revision bump for ampproject#29542
* cl/323858009 Revision bump for ampproject#29409
* cl/323889989 n/a
* cl/324641097 Implements document level byte count limitation.
* cl/324911894 Remove AMP Actions from Validator
* cl/324915915 Revision bump for ampproject#29642

Co-authored-by: Greg Grothaus <greggrothaus@google.com>
Co-authored-by: Amaltas Bohra <amaltas@google.com>
Co-authored-by: honeybadgerdontcare <sedano@google.com>
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.

5 participants