Using "--exit-zero" instead of "ignore_errors" to propagate flake8 internal errors
Review Request #10048 — Created June 29, 2018 and submitted — Latest diff uploaded
flake8returns non-zero status if it runs successfully and finds issues. This means the tool had to pass
ignore_errors=Truein order to ensure
flake8finding issues wasn't reported as
This would mask actual internal errors however, and would result in the bot reporting that
flake8"passed" in the event of an actual internal error.
Before change ->
flake8passes when an internal error occurs.
After change ->
flake8errors with "internal error".