-
Notifications
You must be signed in to change notification settings - Fork 106
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
[Instrumentation] Replace logging "framework" by nim-chronicles #38
Labels
enhancement
New feature or request
Comments
arnetheduck
added a commit
that referenced
this issue
Aug 23, 2018
This blindly changes logging to nim-chronicles - issues that ensue: * absurd amounts of logs in gas computation, slows down tests by orders of magnitude (lacking trace level) - the problem is further exacerbated by chronicles' default multi-line output * unclear if logScope is practical - for example, since vm is split over many files, topics get lost when using simple top-level per-module topics * when passing named object around, scope should incliude the name of the object but this is caught neither by logScope nor by dynamicLogScope
arnetheduck
added a commit
that referenced
this issue
Aug 31, 2018
replace logging module with nim-chronicles (fixes #38)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Current:
Future:
The text was updated successfully, but these errors were encountered: