Skip to content

✨ Playwire AMP-AD implementation #40437

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

Merged
powerivq merged 5 commits into from
Dec 1, 2025

fix linting errors

5cd5827
Select commit
Loading
Failed to load commit list.
Merged

✨ Playwire AMP-AD implementation #40437

fix linting errors
5cd5827
Select commit
Loading
Failed to load commit list.
AMP bundle-size / ampproject/bundle-size succeeded Dec 1, 2025 in 4m 50s

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: 5cd5827
Base commit: 2055896
Code changes: e82f83c

Auto-approved bundle size changes

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

Bundle sizes missing from this PR

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