[PATCH] D53953: [clang-tidy] Get ClangTidyContext out of the business of storing diagnostics. NFC

Fri Nov 2 02:43:57 PDT 2018

sammccall marked an inline comment as done.
sammccall added a comment.

In https://reviews.llvm.org/D53953#1285244, @hokein wrote:

> I think this is a bug, we should return non-zero code when processing a non-existing file. The new behavior seems better to me, though we need to polish the error message.

Thanks. See https://reviews.llvm.org/D53958 for (most of) the needed error message improvements.

Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D53953