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.

Sign up for GitHub

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

Jump to bottom

Simplify logging #225

Open
mdiepart opened this issue Dec 6, 2022 · 0 comments
Open

Simplify logging #225

mdiepart opened this issue Dec 6, 2022 · 0 comments
Assignees
@peterus
Labels
enhancement New feature or request

Comments

@mdiepart
Copy link
Contributor

mdiepart commented Dec 6, 2022

I think that the current way of logging is sub-optimal because we have to pass a System class every time we want to log something and the command to print a log statement is ridiculously long, makign debugging quite difficult.

It would be nice to have a simple header that we can include to have access to all the log commands (info, debug, warn,...)

However, I saw that several libraries are tied to the current way of logging (System and logging libraries are two PlatformIO libraries that are not on this repository but used mainly for this project). I do not know how we could do this in such a way that we keep a consistency between the different versions ?

It would be nice to discuss about that here. If needed I can of course help doing this.
@mdiepart mdiepart added the enhancement New feature or request label Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@peterus peterus

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@peterus @mdiepart