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

Support Windows 2016 #291

Merged
merged 7 commits into from
Feb 7, 2017
Merged

Conversation

gdavison
Copy link

@gdavison gdavison commented Feb 6, 2017

AWS has changed a number of characteristics of Windows Server 2016 instances:

  • the AMI name pattern is now Windows_Server-2016-English-<Style>-*, where <Style> is one of Full-Base for regular servers, Nano-Base for Nano servers, or Full-Containers for server with containers
  • The initialization process uses EC2Launch instead of EC2Config

http://docs.aws.amazon.com/AWSEC2/latest/WindowsGuide/windows-ami-version-history.html#win2k16-amis

@@ -8,7 +8,7 @@ gem "winrm-fs"
gem "activesupport", "~> 4.0"

group :test do
gem "rake"
gem "rake", "< 12"
Copy link
Contributor

Choose a reason for hiding this comment

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

Any reason for this particular pin?

Copy link
Author

Choose a reason for hiding this comment

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

Rake 12 isn't compatible with rubocop 0.28, which is pinned by finstyle 1.4.0.

And the gemspec file says

style and complexity libraries are tightly version pinned as newer releases may introduce new and undesireable style choices which would be immediately enforced in CI

Copy link
Author

Choose a reason for hiding this comment

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

I just tried updating to finstyle 1.5.0. It still isn't compatible with rake 12, and it picks up 11 more style offences :)

Copy link
Contributor

Choose a reason for hiding this comment

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

We're moving away from finstyle and I didn't realize this repo was still on it so new things to do

.travis.yml Outdated
- 2.0.0
- 2.1
- 2.2
- 2.1.8
Copy link
Contributor

Choose a reason for hiding this comment

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

@cheeseplus
Copy link
Contributor

Aside from the inline comments this look great!

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