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

Require Ruby 2.3+ / InSpec 1.47+ and update dev deps #221

Merged
merged 14 commits into from
Jan 23, 2019
Merged

Require Ruby 2.3+ / InSpec 1.47+ and update dev deps #221

merged 14 commits into from
Jan 23, 2019

Conversation

tas50
Copy link
Contributor

@tas50 tas50 commented Jan 19, 2019

Require a supported Ruby release and bump all the deps to current.

tas50 added 7 commits January 18, 2019 23:27
We use expeditor now

Signed-off-by: Tim Smith <[email protected]>
This does nothing, but break things on bundler 2.x

Signed-off-by: Tim Smith <[email protected]>
Remove support for EOL Ruby

Signed-off-by: Tim Smith <[email protected]>
We're not using this anymore. Stop carrying around the baggage.

Signed-off-by: Tim Smith <[email protected]>
Bring our deps up to date.

Signed-off-by: Tim Smith <[email protected]>
No one likes hidden files.

Signed-off-by: Tim Smith <[email protected]>
Since we ship kitchen-inspec in DK for 99.9% of users they already have a fully up-to-date InSpec. This just allows us to remove a conditional and a test.

Signed-off-by: Tim Smith <[email protected]>
@tas50 tas50 changed the title Require Ruby 2.3+ and update dev deps Require Ruby 2.3+ / InSpec 1.47+ and update dev deps Jan 19, 2019
tas50 added 7 commits January 18, 2019 23:41
We need these gems for testing in Travis

Signed-off-by: Tim Smith <[email protected]>
Signed-off-by: Tim Smith <[email protected]>
The extra `|` was catching everything

Signed-off-by: Tim Smith <[email protected]>
Also disable sudo since they ignore this config now.

Signed-off-by: Tim Smith <[email protected]>
Make sure we test on everything we support

Signed-off-by: Tim Smith <[email protected]>
@@ -134,13 +134,13 @@

- \[QUESTION\] kitchen-inspec formatter [\#147](https://github.com/inspec/kitchen-inspec/issues/147)
- Executing inspec against a container from within the host [\#145](https://github.com/inspec/kitchen-inspec/issues/145)
- Option to specify path to Inspec test with kitchen-inspec [\#139](https://github.com/inspec/kitchen-inspec/issues/139)
- Option to specify path to InSpec test with kitchen-inspec [\#139](https://github.com/inspec/kitchen-inspec/issues/139)
Copy link

@jerryaldrichiii jerryaldrichiii Jan 23, 2019

Choose a reason for hiding this comment

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

I ❤️you so much for these changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I worked for a company that changed the case of their name to be all lowercase (so cool). I spent years slowly fixing it all.

Copy link

@jerryaldrichiii jerryaldrichiii left a comment

Choose a reason for hiding this comment

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

I love this so much. Capital S is best S.

Copy link
Contributor

@miah miah left a comment

Choose a reason for hiding this comment

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

Thank you @tas50 !

@tas50 tas50 merged commit 6cfd25f into master Jan 23, 2019
@chef-ci chef-ci deleted the updates branch January 23, 2019 21:23
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.

3 participants