Motivated by #33725 (comment) this continues the effort to enforce iwyu warnings.
This is only adding the compat dir to the CI script since it seems to already pass the iwyu rules.
Including compat into the CI will keep that part of the codebase hygienic
I can close this if it isn't needed or necessary.
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
See the guideline for information on the review process. A summary of reviews will appear here.
<!--174a7506f384e20aa4161008e828411d-->
Reviewers, this pull request conflicts with the following ones:
src/script and treat them as errors by BrandonOdiwuor)If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
Thx, but I don't think it is useful to create conflicting pull requests with already open ones, when the diff is a single line change and that single line conflicts.
It would be better to instead review the existing pull request.
I am going to close this for now. You can pick this up again when the other one is closed or merged, and when this is still relevant after it is closed/merged.
Thx, but I don't think it is useful to create conflicting pull requests with already open ones, when the diff is a single line change and that single line conflicts.
It would be better to instead review the existing pull request.
I am going to close this for now. You can pick this up again when the other one is closed or merged, and when this is still relevant after it is closed/merged.
Gotcha, no problem