Include what you use for library and tests
Open
@syclik is already working on this.
Since Aug 18, 2016.
code cleanup
feature
- Dominant language
- C++
- Stars
- 839
- Forks
- 220
- Avg merge
- 2d 4h
- Merged PRs (30d)
- 14
Description
Summary:
We need to review all of the code and remove all the overincludes.
For tests, it's not clear to me that we want to have minimal includes. The advantage of having the broad library include is that it checks interactions; the advantage of including what you use is that it shows you what needs to be included and will be a whole lot faster.
I missed some tests when I fixed #236.
Description:
Some of the tests and most of the header files include more headers than just the one it needs.
Current Version:
v2.9.0
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.