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

gce-image version bump and documentation update. #1442

Merged
merged 6 commits into from
Jul 19, 2016

Conversation

Gormartsen
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 19, 2016
@@ -390,6 +390,8 @@ var gce = gcloud.compute({
var zone = gce.zone('us-central1-a');
var name = 'ubuntu-http';

// Optionaly you can specify `{ os: 'your-project-id-or-name/ubuntu'}`

This comment was marked as spam.

@Gormartsen
Copy link
Contributor Author

@stephenplusplus done.

@stephenplusplus
Copy link
Contributor

Looks like some line is still over 80: https://travis-ci.org/GoogleCloudPlatform/gcloud-node/jobs/145946346#L917

You can npm run lint locally.

@Gormartsen
Copy link
Contributor Author

Ups. Will update in a hour.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 895b83f on Zen-CI:master into 0292872 on GoogleCloudPlatform:master.

@coveralls
Copy link

coveralls commented Jul 19, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 895b83f on Zen-CI:master into 0292872 on GoogleCloudPlatform:master.

@stephenplusplus stephenplusplus merged commit 6cbbc37 into googleapis:master Jul 19, 2016
@stephenplusplus
Copy link
Contributor

Perfecto. Thanks again 🍻

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.

None yet

4 participants