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.

Sign up for GitHub

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

Jump to bottom

Refactor the Read Config File Logic out of the StartMonitors function #71

Closed

CharityKathure opened this issue Aug 8, 2022 · 0 comments · Fixed by #73

Assignees

Projects

Windows Container Tools

Contributor

CharityKathure commented Aug 8, 2022 •

edited

Currently, the logic to read and convert the config file is part of the StartMonitors function.

This should a separate function, and unit tests should be created to test the new function.

CharityKathure mentioned this issue

Convert utf8 encoded configuration file #70

Merged

CharityKathure self-assigned this

CharityKathure linked a pull request

that will close this issue

Refactor StartMonitors #73

Merged

CharityKathure added this to In progress in Windows Container Tools

CharityKathure closed this as completed in #73

Windows Container Tools automation moved this from In progress to Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment