Closed melvo closed 1 year ago
I don't get any changes when running clang-format-15 -i src/client/QXmppFileSharingManager.cpp
on master. Which version of clang-format are you using?
Base: 68.51% // Head: 68.51% // Increases project coverage by +0.00%
:tada:
Coverage data is based on head (
8002cf6
) compared to base (0cd975e
). Patch has no changes to coverable lines.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
I don't get any changes when running
clang-format-15 -i src/client/QXmppFileSharingManager.cpp
on master. Which version of clang-format are you using?
I used clang-format-14
.
This looks wrong. And in general, you only have to format your personal changes, not the whole project. I can format the whole project when releasing or once a year, so we don't get too much noise from me forgetting to run clang format. A CI check for PRs could also be helpful.
A CI check for PRs could also be helpful.
I opened #491 for that purpose.
@lnjX This should be reopened since both https://github.com/qxmpp-project/qxmpp/pull/491/files#diff-e8fbcf424898cae6fc11412c52bbd57a6bd05561b745e4e07134155327fd402d and my now updated local clang-format-15 reformat it that way.
I cannot reopen this (the branch was recreated)
It is fixed by 98bfe936e25c462df8e3650a436d9db31b237cd9.
PR check list:
\since QXmpp 1.X
,QXMPP_EXPORT
-DBUILD_DOCUMENTATION=ON
)doc/doap.xml
clang-format -i src/<edited-file(s)> tests/<edited-file(s)>