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

Hotfix/config gen #188

Closed
wants to merge 2 commits into from
Closed

Conversation

NamedJason
Copy link
Contributor

I changed the regex to be more discriminating when looking for the $cfg line, so that it will be more likely to find the correct line from each test and thus be able to determine where to store that test's results in the generated config file.

@brianbunke
Copy link
Contributor

Hey! Thanks for following up on your last PR with this.

I've been tracking this in #178, and ended up working on it over the weekend. I just got to submitting #191 to use the AST instead of RegEx, which should achieve the same goal.

I try to assign open issues where I can, because I feel really bad when we spend our free time duplicating work. Sorry! 🙁

@brianbunke brianbunke closed this Oct 10, 2017
@NamedJason
Copy link
Contributor Author

No worries - I saw that it was assigned to you before I started on this. I just wanted to put together something quick to allow me to test a new version of the host vds membership test, so figured that I'd do the quick fix and just change the regex while you worked on a real solution. Most of the time that I spent on it was just familiarizing myself with how the various pieces of vester fit together, which was time well spent even though this commit was obviated. I've never even heard of AST before, so this is a great learning experience for me in several ways!

@brianbunke
Copy link
Contributor

Awesome, glad to hear that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants