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
Thanks for the report @zdm! This was due to our new debug log behavior which opens a file descriptor for the debug log at the beginning of the process. We also attempt to clean out old log files, so what ended up happening was we were attempting to close a currently open file which causes that error on Windows. This is actually ok, so the change was just to not bubble up that error to the warn loglevel and instead drop it to the silly level.
This change should go out on Thursday, but let me know if you see any other weird behavior before that.
Activity
fix: dont warn on error cleaning individual log files
lukekarrys commentedon Dec 7, 2021
Thanks for the report @zdm! This was due to our new debug log behavior which opens a file descriptor for the debug log at the beginning of the process. We also attempt to clean out old log files, so what ended up happening was we were attempting to close a currently open file which causes that error on Windows. This is actually ok, so the change was just to not bubble up that error to the
warnloglevel and instead drop it to thesillylevel.This change should go out on Thursday, but let me know if you see any other weird behavior before that.
zdm commentedon Dec 7, 2021
The problem is that this warnings are mixed with the command output and prevents to read it.
fix: dont warn on error cleaning individual log files
fix(ci-electron): use npm pkg set (no lifecycle) to bump version
docs(ci-electron): record 7.0/7.1/7.2 verification results
robertcsakany