You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 22, 2024. It is now read-only.
meanwhile script.jackett logger also gets the same logger. And now we adding second handler to it.
I think it causes double printing.
More over if we are already using elementum.provider.log why creating one more logger it looks not DRY enough :)
I will revert changes. All this need to be done in separate PR.
What i was trying to achieve is removing this annoying messages "Skipped 1 duplicate messages..":
As i understand the reason is:
in some cases we are using
from elementum.provider import log
and it's calls elementum plugin code:
https://github.com/vasilky3/plugin.video.elementum/blob/master/resources/site-packages/elementum/logger.py
meanwhile script.jackett logger also gets the same logger. And now we adding second handler to it.
I think it causes double printing.
More over if we are already using
elementum.provider.log
why creating one more logger it looks not DRY enough :)I will revert changes. All this need to be done in separate PR.
Originally posted by @vasilky3 in #21 (comment)
The text was updated successfully, but these errors were encountered: