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
With the default parameters, after a colcon build finishes, the powershell window gets stuck on Summary: 1 package finished for a couple of seconds before giving control back to the user.
I noticed this was because of the time.sleep(5) on Line 90 here:
'Failed to show the notification: {e}'.format_map(locals()))
else:
# wait a while before destroying the window
time.sleep(5)
finally:
win32gui.DestroyWindow(hwnd)
Is there a reason to destroy the window? I commented out the line that destroys the window and the notification disappeared after a couple of seconds, and nothing seemed to go wrong.
If destroying the window is required, would it be possible to do this without having the sleep?
The text was updated successfully, but these errors were encountered:
With the default parameters, after a
colcon build
finishes, the powershell window gets stuck onSummary: 1 package finished
for a couple of seconds before giving control back to the user.I noticed this was because of the
time.sleep(5)
on Line 90 here:colcon-notification/colcon_notification/desktop_notification/win32.py
Lines 78 to 92 in 5401d06
Is there a reason to destroy the window? I commented out the line that destroys the window and the notification disappeared after a couple of seconds, and nothing seemed to go wrong.
If destroying the window is required, would it be possible to do this without having the sleep?
The text was updated successfully, but these errors were encountered: