For a cpp file foo.cpp, our style guide instructs folks to put the header foo.hpp at the top of the include list: https://github.com/apache/mesos/blob/master/docs/c%2B%2B-style-guide.md#order-of-includes
This is consistent with Google style guide but in reality most of the our files follow the rule of "treat foo.hpp the same way as other project headers". Since this rule has been introduced to the Mesos style guide I haven't seen much adoption. Most committers still observe the "old" rule when creating new files and have been instructing contributors to do the same. Given the current status I'd suggest we revert the style guide on this? It's only adding confusion and I don't see the need to do a sweeping change to the codebase to comply with the new rule. Thoughts? Yan