[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