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

Add support for regional Google compute disk #164

Merged

Conversation

walterdolce
Copy link
Contributor

Signed-off-by: Walter Dolce walterdolce@gmail.com

Description

This PR adds support for verifying the state of regional disks.

Issues Resolved

#163

PS: changing the table in the README is a nightmare 😅

@walterdolce walterdolce requested a review from a team as a code owner August 21, 2019 14:40
@walterdolce walterdolce force-pushed the feature/add-support-for-regional-disks branch from 28f3216 to 21e2214 Compare August 21, 2019 14:41
Signed-off-by: Walter Dolce <walterdolce@gmail.com>
slevenick pushed a commit to slevenick/inspec-gcp that referenced this pull request Aug 22, 2019
Signed-off-by: Modular Magician <magic-modules@google.com>
@skpaterson
Copy link

Thanks for this @walterdolce ! To resolve the build failure, rebasing against master and making the following minor change should allow us to proceed e.g.

libraries/google_compute_regional_disk.rb:1:1: C: Style/FrozenStringLiteralComment: Missing magic comment # frozen_string_literal: true.

Apologies for the pain editing the markdown table, I generate this automatically based on the files in the docs directory. Should probably commit that script somewhere and update the docs...

@walterdolce
Copy link
Contributor Author

walterdolce commented Aug 23, 2019

Missing magic comment

A comment that causes a build to fail? 🤔 Is that expected? I'll do as instructed in the meantime.

Should probably commit that script somewhere and update the docs...

That would be great 😄

Signed-off-by: Walter Dolce <walterdolce@gmail.com>
Signed-off-by: Walter Dolce <walterdolce@gmail.com>
@walterdolce
Copy link
Contributor Author

@skpaterson all done!

@skpaterson
Copy link

Awesome, thanks @walterdolce.

Copy link

@skpaterson skpaterson left a comment

Choose a reason for hiding this comment

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

👍

@skpaterson skpaterson merged commit f6cd5f1 into inspec:master Aug 23, 2019
skpaterson pushed a commit that referenced this pull request Aug 23, 2019
Signed-off-by: Modular Magician <magic-modules@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants