Skip to content

Improve logging #108

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

Merged
merged 3 commits into from
Jun 1, 2023
Merged

Conversation

tnull
Copy link
Collaborator

@tnull tnull commented May 23, 2023

Based on #93, Fixes #109.

We improve our logging output by logging more detailed errors and introduce logging to start/stop methods.

Additionally, we expose the log level via Config::log_level and Builder::set_log_level.

@tnull tnull force-pushed the 2023-05-improve-logging branch from 6b772ae to 5181b30 Compare May 23, 2023 11:31
@tnull tnull force-pushed the 2023-05-improve-logging branch 2 times, most recently from b022172 to 4266b2f Compare May 23, 2023 14:05
@tnull tnull mentioned this pull request May 23, 2023
47 tasks
@tnull tnull added this to the 0.1 milestone May 23, 2023
@tnull tnull force-pushed the 2023-05-improve-logging branch 4 times, most recently from af8de05 to f2d157f Compare May 23, 2023 20:02
@tnull
Copy link
Collaborator Author

tnull commented May 23, 2023

Rebased after #93 has been merged.

@tnull tnull force-pushed the 2023-05-improve-logging branch from f2d157f to d80aceb Compare May 25, 2023 09:57
@tnull
Copy link
Collaborator Author

tnull commented May 25, 2023

Rebased on main after #101 landed.

@tnull tnull force-pushed the 2023-05-improve-logging branch from d80aceb to 3d87eae Compare May 27, 2023 06:47
@tnull
Copy link
Collaborator Author

tnull commented May 27, 2023

Rebased on main after #88 landed.

@tnull tnull force-pushed the 2023-05-improve-logging branch from 3d87eae to f4d3ca8 Compare May 27, 2023 06:54
@jkczyz jkczyz self-requested a review June 1, 2023 13:10
@tnull tnull force-pushed the 2023-05-improve-logging branch from f4d3ca8 to 31fd0c8 Compare June 1, 2023 13:45
Copy link
Contributor

@jkczyz jkczyz left a comment

Choose a reason for hiding this comment

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

LGTM. Please squash.

@tnull tnull force-pushed the 2023-05-improve-logging branch from 31fd0c8 to 7ece643 Compare June 1, 2023 15:51
@tnull
Copy link
Collaborator Author

tnull commented Jun 1, 2023

LGTM. Please squash.

Squashed without further changes.

@tnull tnull force-pushed the 2023-05-improve-logging branch from 7ece643 to 52ccd58 Compare June 1, 2023 19:34
@tnull tnull merged commit 427e74c into lightningdevkit:main Jun 1, 2023
@tnull tnull mentioned this pull request Jun 1, 2023
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.

Expose easy way to control log level
2 participants