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

feat: update logging and output functionality #82

Merged
merged 3 commits into from
Mar 13, 2024

Conversation

rhizoome
Copy link

@rhizoome rhizoome commented Mar 5, 2024

Multiple commits not all related to logging and output functionality

@rhizoome rhizoome force-pushed the jlf/feat-update-logging-and-output-functionality branch 2 times, most recently from c2d3efe to 550e15e Compare March 5, 2024 13:05
@rhizoome rhizoome force-pushed the jlf/feat-update-logging-and-output-functionality branch from 2c55c29 to 35bf720 Compare March 13, 2024 12:11
@rhizoome rhizoome force-pushed the jlf/feat-update-logging-and-output-functionality branch 3 times, most recently from 7235b6e to 0f729bd Compare March 13, 2024 12:29
@rhizoome rhizoome force-pushed the jlf/feat-update-logging-and-output-functionality branch 2 times, most recently from 1374c2e to d755fbc Compare March 13, 2024 12:46
@rhizoome rhizoome marked this pull request as ready for review March 13, 2024 12:56
@rhizoome rhizoome force-pushed the jlf/feat-update-logging-and-output-functionality branch 2 times, most recently from 951a05b to b366bd6 Compare March 13, 2024 13:11
@rhizoome rhizoome requested review from winged and Melkor333 March 13, 2024 13:13
Copy link
Member

@winged winged left a comment

Choose a reason for hiding this comment

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

LGTM! Just a tiny suggestion, but you may ignore it, as it's probably not worth it

Comment on lines 30 to 33
if util.isatty():
formats = self.FORMATS_COLOR # pragma: no cover
else:
formats = self.FORMATS
Copy link
Member

Choose a reason for hiding this comment

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

Could we move this into __init__()? Then you'd only have to make the decision once

@rhizoome rhizoome merged commit 828bd66 into main Mar 13, 2024
1 check passed
@rhizoome rhizoome deleted the jlf/feat-update-logging-and-output-functionality branch March 21, 2024 01:25
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