Skip to content
This repository was archived by the owner on Sep 4, 2020. It is now read-only.
This repository was archived by the owner on Sep 4, 2020. It is now read-only.

Improve New-VesterConfig $Desired RegEx #178

Open
@brianbunke

Description

@brianbunke

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 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.)

Your Environment

#176

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions