You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.
New-VesterConfig successfully pulls the config value to populate from the $Desired line.
Current Behavior
The RegEx catches all instances of $cfg, instead of just the one desired. (Heyooo puns)
Possible Solution
Make dirty RegEx less dirty and more complicated
Be awesome and use the AST to properly parse the variable assignment
Steps to Reproduce (for bugs)
Try to run New-VesterConfig against the #176 version of the module, and watch it not populate a value for the new host VDS membership test. (Because "$cfg" is commented out in the fix block.)
Expected Behavior
New-VesterConfig
successfully pulls the config value to populate from the$Desired
line.Current Behavior
The RegEx catches all instances of
$cfg
, instead of just the one desired. (Heyooo puns)Possible Solution
Make dirty RegEx less dirty and more complicatedBe awesome and use the AST to properly parse the variable assignment
Steps to Reproduce (for bugs)
Try to run
New-VesterConfig
against the #176 version of the module, and watch it not populate a value for the new host VDS membership test. (Because "$cfg
" is commented out in the fix block.)Your Environment
#176
The text was updated successfully, but these errors were encountered: