Should fail of gcloud fail

[HazemKhaled]

Currently shell.exec('gcloud') error not handled, so whatever happens while deploy it give me success instead of fail, so my CI deployment trigger never work.

Example

ERROR: (gcloud.app.deploy) Permissions error fetching application [apps/my-project]. Please make sure you are using the correct project ID and that you have permission to view applications on the project.
Build finished successfully!.

Another example

ERROR: (gcloud.app.deploy) User [my-project@appspot.gserviceaccount.com] does not have permission to access app [my-project] (or it may not exist): App Engine Admin API has not been used in project 934221695787 before or it is disabled. Enable it by visiting https://console.developers.google.com/apis/api/appengine.googleapis.com/overview?project=1234567890 then retry. If you enabled this API recently, wait a few minutes for the action to propagate to our systems and retry.
- '@type': type.googleapis.com/google.rpc.Help
  links:
  - description: Google developers console API activation
    url: https://console.developers.google.com/apis/api/appengine.googleapis.com/overview?project=934221695787
Build finished successfully!.

[HazemKhaled]

I think it should be handled here, I'll try to create pull request soon
https://github.com/EducationLink/meteor-google-cloud/blob/master/src/lib/google.js#L92

[raphaelarias]

That's great, makes perfect sense. Pull requests are welcomed, otherwise it will add error handling. I'm going to release a new version soon, we can include this improvement.