So I decided that we should put some rigorous tests into our projects that would make sure that at a very basic level the public headers of libraries would be possible to use "out of the box". Starting with this project. (And then I plan to open very similar PRs into all our other projects as well.)
Technically some of the headers that these tests pick up are not strictly "public headers". But in the end any header that's visible to the users should be possible to include by itself. (At least currently we don't have any shenanigans in this project that would prevent us from doing this.)
As you can see, the tests did find one missing include even in this project. :wink:
While working on https://github.com/acts-project/traccc/pull/328 I had to find the Detray's headers are in a bit of a disrepair at the moment.. :frowning:
So I decided that we should put some rigorous tests into our projects that would make sure that at a very basic level the public headers of libraries would be possible to use "out of the box". Starting with this project. (And then I plan to open very similar PRs into all our other projects as well.)
Technically some of the headers that these tests pick up are not strictly "public headers". But in the end any header that's visible to the users should be possible to include by itself. (At least currently we don't have any shenanigans in this project that would prevent us from doing this.)
As you can see, the tests did find one missing include even in this project. :wink: