Skip to content

Validator rollup#25559

Merged
banaag merged 4 commits intoampproject:masterfrom
banaag:validator-rollup
Nov 12, 2019
Merged

Validator rollup#25559
banaag merged 4 commits intoampproject:masterfrom
banaag:validator-rollup

Conversation

@banaag
Copy link
Copy Markdown
Contributor

@banaag banaag commented Nov 12, 2019

@amp-owners-bot
Copy link
Copy Markdown

amp-owners-bot bot commented Nov 12, 2019

Hey @ampproject/wg-caching, these files were changed:

  • extensions/amp-audio/0.1/test/validator-amp-audio.html
  • extensions/amp-audio/0.1/test/validator-amp-audio.out
  • validator/validator-main.protoascii

Copy link
Copy Markdown
Contributor

@honeybadgerdontcare honeybadgerdontcare left a comment

Choose a reason for hiding this comment

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

please revert the two .out files that duplicate content, otherwise lgtm

@banaag
Copy link
Copy Markdown
Contributor Author

banaag commented Nov 12, 2019

@googlebot

@banaag banaag merged commit b203bba into ampproject:master Nov 12, 2019
@banaag banaag deleted the validator-rollup branch November 12, 2019 20:19
micajuine-ho pushed a commit to micajuine-ho/amphtml that referenced this pull request Dec 27, 2019
* cl/279189628 When heights or sizes is empty, the layout should be fixed

* cl/279442666 Revision bump for ampproject#25339

* Removed unwanted files from pull request
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.

3 participants