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
Would it be possible to improve the error messages caused when isort fails to explain the problem? I would normally expect a warning not to fail an action.
When running isort, we expect it to return a diff. However, if any other message is sent, we will treat it as a failure and discard all changes.
The message can be anything, in the case of #12949 and #13948 (comment) it is just a warning, which isn't expected to cause isort to fail.
The text was updated successfully, but these errors were encountered:
kimadeline
changed the title
Do not discard the diff and report a failure when isort outputs a warning message
When isort outputs a warning message, do not discard the diff and report a failure
Sep 23, 2020
from @Lexicality in #13948 (comment):
When running
isort
, we expect it to return a diff. However, if any other message is sent, we will treat it as a failure and discard all changes.The message can be anything, in the case of #12949 and #13948 (comment) it is just a warning, which isn't expected to cause
isort
to fail.The text was updated successfully, but these errors were encountered: