Skip to content

Conversation

kylesayrs
Copy link
Contributor

@kylesayrs kylesayrs commented Sep 9, 2025

Purpose

  • Support loguru logging, which supports features such as logging once

Changes

  • Add logging configuration
  • Ensure that logging configuration occurs upon module import

Testing

Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>
Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>
Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>
@kylesayrs kylesayrs changed the title [Logging] Support use of loguru [Logging] Support use of loguru, fix quality Sep 9, 2025
Copy link
Contributor

@fynnsu fynnsu left a comment

Choose a reason for hiding this comment

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

Looks good but added a few suggestions.

Copy link
Contributor

@brian-dellabetta brian-dellabetta left a comment

Choose a reason for hiding this comment

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

LGTM, pending Fynn's comments

Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>
@kylesayrs kylesayrs changed the title [Logging] Support use of loguru, fix quality [Logging] Support use of loguru Sep 9, 2025
fynnsu
fynnsu previously approved these changes Sep 9, 2025
* better type hints, warn once

Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>

* remove unneeded import

Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>

* use warnings

Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>

---------

Signed-off-by: Kyle Sayers <kylesayrs@gmail.com>
@kylesayrs kylesayrs dismissed stale reviews from fynnsu and brian-dellabetta via 7391fc4 September 10, 2025 13:34
@kylesayrs kylesayrs enabled auto-merge (squash) September 11, 2025 18:25
@kylesayrs kylesayrs merged commit 891da51 into main Sep 11, 2025
2 checks passed
@kylesayrs kylesayrs deleted the kylesayrs/loguru branch September 11, 2025 18: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.

3 participants