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

Add logging module #53

Merged
merged 8 commits into from
Oct 31, 2023
Merged

Add logging module #53

merged 8 commits into from
Oct 31, 2023

Conversation

YHordijk
Copy link
Contributor

Added a logging module to TCutility that can be used later to form some nicely formatted outputs/summaries.
I also made some formatting changes to test-cases, but I accidentally pushed them to this branch, woops!

@YHordijk YHordijk added enhancement New feature or request testing Issues for improving testing labels Oct 29, 2023
@YHordijk YHordijk linked an issue Oct 29, 2023 that may be closed by this pull request
Copy link
Contributor

@SiebeLeDe SiebeLeDe left a comment

Choose a reason for hiding this comment

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

See my comment about logging

TCutility/log.py Show resolved Hide resolved
…o take a level argument which will be compared to module-level log_level variable. If it is greater or equal it will print the message, else it will not print it.
@YHordijk YHordijk requested a review from SiebeLeDe October 31, 2023 08:13
Copy link
Contributor

@SiebeLeDe SiebeLeDe left a comment

Choose a reason for hiding this comment

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

Nice addition of log levels

@SiebeLeDe SiebeLeDe merged commit d2f270c into main Oct 31, 2023
6 checks passed
@SiebeLeDe SiebeLeDe deleted the add-logging-module branch October 31, 2023 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request testing Issues for improving testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add logging module
2 participants