Skip to content

Conversation

@fhinkel
Copy link
Contributor

@fhinkel fhinkel commented Nov 8, 2018

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 8, 2018
@fhinkel
Copy link
Contributor Author

fhinkel commented Nov 8, 2018

This fixes the linter problems.

Copy link
Contributor

@ace-n ace-n left a comment

Choose a reason for hiding this comment

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

IIRC/FYI: that image may not have (or have an outdated version of) gcloud

@fhinkel
Copy link
Contributor Author

fhinkel commented Nov 8, 2018

@ace-n Thanks for the heads up, these tests should not require gcloud.

@fhinkel
Copy link
Contributor Author

fhinkel commented Nov 8, 2018

functions/sendgrid is failing - but I didn't touch the functions image. Any idea?

@fhinkel
Copy link
Contributor Author

fhinkel commented Nov 8, 2018

Same linter problem. Can I make that test non-blocking for now until you figure out what to do with functions? The problem is that the test is using Node 8 syntax on a Node 6 image.

@fhinkel fhinkel requested a review from ace-n November 8, 2018 04:06
@fhinkel fhinkel merged commit fbbacac into master Nov 8, 2018
@fhinkel fhinkel deleted the useNode8 branch November 8, 2018 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants