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.

Sign up for GitHub

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

Jump to bottom

fixed missing comma #89

Merged

azr merged 1 commit into hashicorp:main from jacobmammoliti:ubuntu_pro_fix

Mar 22, 2022

Contributor

jacobmammoliti commented Mar 22, 2022

Added the missing comma on line 281 causing the following error on go build:

# github.com/hashicorp/packer-plugin-googlecompute/builder/googlecompute
builder/googlecompute/driver_gce.go:281:24: syntax error: unexpected newline, expecting comma or }

Should have validated before initial PR but fixed now... go build builds successfully.


          fixed missing comma

99e3d0f

jacobmammoliti requested a review from a team as a code owner

March 22, 2022 15:28

azr approved these changes

View reviewed changes

Contributor

azr left a comment

Ah, good catch I remember seeing the tests pass, but looks like not !

azr merged commit fe3e12b into hashicorp:main

azr added the tech-debt label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels