-
Notifications
You must be signed in to change notification settings - Fork 62
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include what you use #165
Comments
Hi, Also: I need to specify a clang version for IWYU. Which should I use? |
Hi, wl-clipboard will not use GitHub Actions (as long as it requires adding GitHub-specific files to the repo), so maybe setting up an action is not the optimal way to start. I was rather thinking we'd run IWYU once and act on its output, then maybe check what it has to say every once in a while. |
That being said, your action is using |
Oh ok, may I ask why? I was thinking that If we put it in a |
Oh right, I've found it now. It was just not mentioned anywhere, so I thought there was no package available. |
Regardless of whether or not you want to merge this, it now works on my fork. |
We should run https://include-what-you-use.org/ on wl-clipboard and apply (some of) its suggestions, within reason.
It's easy to use with Meson, see mesonbuild/meson#2637 (comment)
The text was updated successfully, but these errors were encountered: