Michael Jones

Results 151 comments of Michael Jones

Ah, sorry, I should have clarified that for you. Link: https://github.com/include-what-you-use/include-what-you-use/blob/master/docs/IWYUMappings.md A "map" file is basically a file that override's the built in rules for IWYU. For example, *most* compilers...

As for maintenance burden: You're certainly right to be cautious about that. I'm *not* recommending annotating every single file in every single place. Generally IWYU doesn't have too many false-positives....

Oh, one thing to note: My CI example above, requires ``` set_target_properties(${PROJECT_NAME} PROPERTIES EXPORT_COMPILE_COMMANDS TRUE) ``` Or `CMAKE_EXPORT_COMPILE_COMMANDS` to be set to true on the cmake configure step globally. Just...

Ah, here's an example of where entt could make IWYU be less noisy. `` depends on ``, in the sense of the `EnTT::Example` and `EnTT::Dependency` that I described above. Changing...

The YAML snippet i pasted in a previous comment is a full github action script that you can add to your repo. See how i use it here https://github.com/TheOpenSpaceProgram/osp-magnum/blob/master/.github/workflows/analyzers.yml

I gave it a shot, not sure which parts of my changes are strictly needed, so some minimization might be worth doing. https://github.com/jonesmz/entt/tree/iwyu The gtest annoyance i think i fixed...

I have no idea what the reason for the curl error is. I'm just using your ccache-action as-is. I don't otherwise interact with curl in any way.

I'm seeing that the failure is inconsistent. Sometimes (with no changes to the macos.yml) it works, sometimes it doesn.t This makes me think that there is potentially a misconfigured github...

Out of about 6-7 runs, it failed twice with ssl errors. But I didn't have enough data points to say there was much of a pattern.