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

Remove VeeWee definitions from project #358

Merged
merged 4 commits into from
May 20, 2015
Merged

Remove VeeWee definitions from project #358

merged 4 commits into from
May 20, 2015

Conversation

fnichol
Copy link
Contributor

@fnichol fnichol commented May 20, 2015

The remaining VeeWee definitions were exclusively Windows based but were
also suffering from age. The current state of Packer may allow for
WinRM-based templates for Windows in the near future but in the meantime
removing VeeWee simplifies this project's dependencies.

The remaining VeeWee definitions were exclusively Windows based but were
also suffering from age. The current state of Packer may allow for
WinRM-based templates for Windows in the near future but in the meantime
removing VeeWee simplifies this project's dependencies.
@fnichol
Copy link
Contributor Author

fnichol commented May 20, 2015

@schisamo, @yzl, Ready for a review. Kind of big, but I've been building these as a I go the last 2 days and confirming that the templates still build correctly.

@schisamo
Copy link
Contributor

:shipit: 🐑

fnichol added a commit that referenced this pull request May 20, 2015
Remove VeeWee definitions from project
@fnichol fnichol merged commit 0660bcc into master May 20, 2015
@fnichol fnichol removed the Reviewing label May 20, 2015
@fnichol fnichol deleted the deprecate-veewee branch May 20, 2015 22:06
@fnichol fnichol mentioned this pull request May 29, 2015
@tas50 tas50 added Type: Enhancement Adds new functionality. and removed Improvement labels Jan 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Adds new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants