-
Notifications
You must be signed in to change notification settings - Fork 305
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
feat: use windows os evaluation #844
Conversation
06588c9
to
8dd04b9
Compare
e261999
to
b7e306b
Compare
b7e306b
to
2513342
Compare
2513342
to
33aad58
Compare
33aad58
to
2ef6a36
Compare
Adds the option to build the Windows image using the evaluation, **by default**, with the `vm_inst_os_eval` variable. If set to `true` (default), the image is built with evaluation, else it will look for a provided key. Co-Authored-By: Ryan Johnson <ryan@tenthirtyam.org>
2ef6a36
to
bfe7d08
Compare
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.
LGTM and tested! 🚀
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Summary of Pull Request
Adds the option to build the Windows image using the evaluation, by default, with the
vm_inst_os_eval
variable.If set to
true
(default), the image is built with evaluation, else it will look for a provided key.Type of Pull Request
type/bug
type/feature
ortype/enhancement
type/docs
type/refactor
type/chore
Please describe:
Issue Number: N/A
Test and Documentation Coverage
Breaking Changes?