-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Run amp-linter checks as part of this module #1160
Conversation
@@ -0,0 +1 @@ | |||
gcloud functions deploy checkFonts --entry-point demo/index.js --runtime nodejs14 --trigger-http --allow-unauthenticated --project amp-dev-staging |
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.
are we explicitly needing 14 vs latest stable?
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.
No, but you need to pick a runtime and 14 is the latest supported one.
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.
Oh? The docs read like you could just declare nodejs
. 👍
@@ -178,6 +181,7 @@ class PageAnalyzer { | |||
}; | |||
|
|||
return { | |||
url: window.location, |
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.
url
is the location object? not window.location.href
?
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.
good one! fixed
77ad5bc
to
163df7a
Compare
163df7a
to
f712fdd
Compare
No description provided.