[PATCH] D42134: [lnt] Fix global status page being unable to load due to crash
Tim Bradgate via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 17 01:36:28 PST 2018
Tim020 added a comment.
> And out of interest: Are you actually using the /global_status page? `global_status` (and `summary_report` for that matter) is a feature/page that doesn't appear to be used by anyone in our company and it has zero test coverage. Hence the two pages bitrot and are broken a lot. I would love to just remove them (the alternative would be to create proper tests, but I'm not even sure about the typical usecase for those two pages and don't want to spend time creating tests for something that seems unused...)
We are not necessarily using it as a feature, but was nice to get it into a working state :)
Repository:
rL LLVM
https://reviews.llvm.org/D42134
More information about the llvm-commits
mailing list