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
remove IS_AMP_ALT #31493
remove IS_AMP_ALT #31493
Conversation
Hey @jridgewell! These files were changed:
|
🔥 🔥 🔥 🔥 Love this 🔥 🔥 🔥 🔥 🔥 |
* v0.js code. | ||
* @type {boolean|undefined} | ||
*/ | ||
const shouldMainBootstrapRun = !self.IS_AMP_ALT; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to the description, don't we still need this variable for amp-inabox
and amp-shadow
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry late reply. this should have been only needed for single pass, if theres any other description i haven't removed about it, could you point it out? thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
97d67bd
to
b957076
Compare
This pull request introduces 1 alert when merging b957076 into 2f0ac6e - view on LGTM.com new alerts:
|
89e601d
to
9d862ac
Compare
9d862ac
to
5d3a4e9
Compare
This flag was only needed for single pass which is abandoned
5d3a4e9
to
137d6fa
Compare
This flag was only needed for single pass which is abandoned