-
-
Notifications
You must be signed in to change notification settings - Fork 664
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
WparamReturn #4664
WparamReturn #4664
Conversation
It looks like you need to apply clang-format to some of the changed files. |
Error: Code is inconsistent with ITK Coding Style. Add the action:ApplyClangFormat PR label to correct. This is the message generated by clang-format linter. Is it possible to type action:ApplyClangFormat in a comment here? I am working with a different version of clang-format which gives different formatting changes than here. |
I added that label (in GitHub UI on the right), but the action triggered ( I forgot who created clang-format action, @thewtex or someone else (@jhlegarreta?). @blowekamp has a parallel PR #4651 to update clang-format used to the latest version. |
I don't think it was me. Cannot remember if adding the label or calling something like |
Now that you mentioned it, I think it used to work for PRs made from branches in this repository, but not from branches in forks. But I think that the action itself might be broken now. |
I created the Apply Clang Format Action. At the time it was created, it was limited in its ability to apply to PR's from forks because of permission functionality in GitHub Actions. Since then, I think GitHub has added the capability to indicate that an Action can push to a PR, etc. If someone wants to look into the adding the config required, please do. I plan to spend time helping with the pre-commit hook / action setup as a replacement. |
Removed \return when there is no function. Kept the rest of the comment
1aad186
to
412c05b
Compare
Contains the following fixes