-
Notifications
You must be signed in to change notification settings - Fork 59
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
Require InSpec 2.2.64+ to support plugin v2 #254
Conversation
Thanks for pointing this out @satyanash. This fixes #253 Signed-off-by: Tim Smith <[email protected]>
@tas50 manually specifying
So, |
Looks like
|
@satyanash Can you give me a more complete stack trace on that errror? |
@tas50 More info on the setup:
Here you go:
|
@tas50 btw, I am able to get everything working for Ruby
So, most likely something changed in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @tas50 !
The major version boundary is a little odd since the breaking change (needing V2 plugins) was some time ago. I think it is best to bump it though.
@james-stocks and @tas50, first thanks for that patch which properly follows sem-versioning. I think a yank would be too drastic, but a revert for a 1.3.3 could be great :D |
Thanks for pointing this out @satyanash. This fixes #253
Signed-off-by: Tim Smith [email protected]