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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃彈馃毊 Remove legacy bundle-size check #19541

Merged

Conversation

danielrozenberg
Copy link
Member

@danielrozenberg danielrozenberg commented Nov 29, 2018

This PR removes the old bundle-size checks based on .max_size. Bundle sizes will no longer fail Travis builds. Woohoo!

DO NOT MERGE until:

  • AMP bundle-size app has been running successfully for ~2 weeks with no unforeseen issues or unexpected behaviours (last unstable version was fixed on 1/9)
  • AMP bundle-size app has been switched to a blocking check

@danielrozenberg
Copy link
Member Author

/to @erwinmombay @rsimha this is ready to be reviewed and merged 馃帀

@danielrozenberg danielrozenberg changed the title 馃彈馃毊 Remove legacy bundle-size check [DO NOT MERGE see description] 馃彈馃毊 Remove legacy bundle-size check Jan 23, 2019
@danielrozenberg danielrozenberg merged commit ebe9247 into ampproject:master Jan 23, 2019
@danielrozenberg danielrozenberg deleted the remove-legacy-bundle-size branch January 23, 2019 22:41
danielrozenberg added a commit to ampproject/amphtml-build-artifacts that referenced this pull request Jan 23, 2019
danielrozenberg added a commit to ampproject/amphtml-build-artifacts that referenced this pull request Jan 28, 2019
danielrozenberg pushed a commit to ampproject/amphtml-build-artifacts that referenced this pull request Feb 25, 2019
Unused since ampproject/amphtml#19541, and hopefully for reals this time
danielrozenberg added a commit to ampproject/amphtml-build-artifacts that referenced this pull request Feb 25, 2019
Unused since ampproject/amphtml#19541, and hopefully for reals this time
noranazmy pushed a commit to noranazmy/amphtml that referenced this pull request Mar 22, 2019
* Remove legacy bundle-size check

* Remove unnecessary space
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.

None yet

3 participants