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.

Sign up for GitHub

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

Jump to bottom

Adding new vendor IncrementX #39237

Merged
merged 15 commits into from
Jul 25, 2023

Update incrementx.js

e091035
Select commit
Loading
Failed to load commit list.
Merged

Adding new vendor IncrementX #39237

Update incrementx.js
e091035
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
AMP bundle-size / ampproject/bundle-size succeeded Jul 18, 2023 in 11m 7s

No approval necessary

This pull request does not change any brotli bundle sizes by a significant amount, so no special approval is necessary.

Commit details

Head commit: e091035
Base commit: 57b653b
Code changes: a79f0f7

Auto-approved bundle size changes

  • dist/v0/amp-sticky-ad-0.1.js: Δ +0.02KB
  • dist/v0/amp-sticky-ad-1.0.js: Δ +0.02KB
  • dist/v0/amp-ad-0.1.mjs: Δ +0.04KB
  • dist/v0/amp-ad-0.1.js: Δ +0.03KB
  • dist/v0/amp-ad-network-smartadserver-impl-0.1.mjs: Δ +0.02KB
  • dist/v0/amp-ad-network-valueimpression-impl-0.1.mjs: Δ -0.01KB
  • dist/v0/amp-ad-network-smartadserver-impl-0.1.js: Δ +0.11KB
  • dist/v0/amp-ad-network-adsense-impl-0.1.mjs: Δ -0.09KB
  • dist/v0/amp-ad-network-doubleclick-impl-0.1.mjs: Δ +0.03KB
  • dist/v0/amp-ad-network-adsense-impl-0.1.js: Δ -0.03KB
  • dist/amp4ads-v0.mjs: Δ +0.01KB
  • dist/v0/amp-ad-network-doubleclick-impl-0.1.js: Δ +0.06KB
  • dist/amp4ads-v0.js: Δ -0.05KB

Bundle sizes missing from this PR

  • dist.3p/current-min/vendor/incrementx.js: (8.39 KB) missing on the main branch
  • dist.3p/current-min/vendor/incrementx.mjs: (6.74 KB) missing on the main branch
View more details on AMP bundle-size