-
Notifications
You must be signed in to change notification settings - Fork 21
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
dev: remove logging perf of coverage in main #673
Conversation
make it debug level? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
make it debug
ok will set nightly in info and ths in debug |
change into logging total coverage time once |
05ac61b
to
8b20d25
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since this is far from any prio, I wouldn't mind either just putting debug to the logger as discussed earlier, of remove this logger, but no accumulation like that
6800e22
to
fbf4aae
Compare
ok i vote to remove it then, it's so verbose it's unreadable |
le logger de perf de coverage rend la nightly trop verbose
je recommande de supprimer les log info dans le coverage