-
Notifications
You must be signed in to change notification settings - Fork 519
fix: replace some leftover acsengine by aksengine #268
fix: replace some leftover acsengine by aksengine #268
Conversation
Codecov Report
@@ Coverage Diff @@
## master #268 +/- ##
=======================================
Coverage 53.16% 53.16%
=======================================
Files 95 95
Lines 14244 14244
=======================================
Hits 7573 7573
Misses 6006 6006
Partials 665 665 |
/lgtm |
/hold Let's get #253 merged first, as this one will probably be easier to resolve merge conflicts than that one |
2014567
to
912416e
Compare
@bacongobbler I rebased can you please do a quick check to make sure I didn't unintentionally undo some of your doc changes? |
/hold cancel |
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.
Nope! This looks good to me.
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: CecileRobertMichon, jackfrancis, tariq1890 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Reason for Change:
Fix some acs-engine references that were left in the code. Most are variables. One is the
generatorCode
that is used in theCreationSource
tag for VMs.Issue Fixed:
Requirements:
Notes: