Skip to content
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.

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

Allow setting any log level #284

Merged
merged 2 commits into from
Jan 9, 2024
Merged

Conversation

stanleyrh
Copy link
Contributor

It would be useful to set a specific log level by providing a string as other levels apart from info and debug are useful.

Have prioritised the existing debug flag for backwards compatibility but would recommend its removal.

It would be useful to set a specific log level by providing a string as other levels apart from info and debug are useful.

Have prioritised the existing debug flag for backwards compatibility but would recommend its removal.
Admiral-Piett
Admiral-Piett previously approved these changes Dec 8, 2023
Copy link
Owner

@Admiral-Piett Admiral-Piett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Copy link
Owner

@Admiral-Piett Admiral-Piett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change looks good, but looks like the tests pointed out that we need to declare the variable first in case it's not passed in. Mind taking care of that?

@stanleyrh
Copy link
Contributor Author

Ah my bad! Was editing straight in the browser and didn't run the tests 🤦
Have fixed now

@Admiral-Piett
Copy link
Owner

Admiral-Piett commented Jan 9, 2024

@stanleyrh nice! Merging it! Thanks for taking care of that.

@Admiral-Piett Admiral-Piett merged commit a1441eb into Admiral-Piett:master Jan 9, 2024
2 checks passed
@stanleyrh stanleyrh deleted the patch-1 branch February 1, 2024 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants