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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🚮 Disable layers in production #23968

Closed
wants to merge 3 commits into from

Conversation

jridgewell
Copy link
Contributor

@jridgewell jridgewell commented Aug 14, 2019

This disables layers unless you've locally built the AMP codebase (for manual testing) or are running in a test environment (automatic testing).

Pending our go/layers-go-nogo decision.

Copy link

@dreamofabear dreamofabear left a comment

Choose a reason for hiding this comment

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

Successful in 13m — Δ +0.03KB

Looks like it's still not getting DCE'ed.

@jridgewell
Copy link
Contributor Author

Depends on #24027 and #24028

This disables layers unless you've locally built the AMP codebase (for manual testing) or are running in a test environment (automatic testing).
@jridgewell
Copy link
Contributor Author

Closing for #24094.

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.

3 participants