-
Notifications
You must be signed in to change notification settings - Fork 148
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 a debugging log #461
Comments
Would using |
Yes, For the non-null logger, we can extend |
- Add PSR/Log as dependency - Add a new SimpleLogger - Use the NullLogger per default Helps with MyIntervals#461 Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>
- Add PSR/Log as dependency - Add a new SimpleLogger - Use the NullLogger per default Helps with MyIntervals#461 Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>
- Add PSR/Log as dependency - Add a new SimpleLogger - Use the NullLogger per default Helps with MyIntervals#461 Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>
- Add PSR/Log as dependency - Add a new SimpleLogger - Use the NullLogger per default Helps with MyIntervals#461 Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>
- Add PSR/Log as dependency - Add a new SimpleLogger - Use the NullLogger per default Helps with MyIntervals#461 Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>
- Add PSR/Log as dependency - Add a new SimpleLogger - Use the NullLogger per default Helps with MyIntervals#461 Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>
For every error that currently is throwing an exception or that is worked around using the lenient mode, we should log the problem, the line number and the corresponding CSS fragment (where it makes sense).
We should log this using a PSR-compatible logger and provide a simple default logger to
Parser
. It should be possible to override the logger from the outside (so people can plug in their own logging if they like).We'll also need to
psr/log
to our dependencies.It should also be possible to disable debugging in order to improve performance.
The text was updated successfully, but these errors were encountered: