diff options
author | Alexander Lohnau <alexander.lohnau@gmx.de> | 2022-04-30 08:57:26 +0200 |
---|---|---|
committer | Alexander Lohnau <alexander.lohnau@gmx.de> | 2022-05-30 17:24:13 +0200 |
commit | 79caa00d0949138b237ecced794a903237bcf2b5 (patch) | |
tree | 08c7f36936997ad6915cbce7ac12b29390ea1adc /docs/module/ECMQmlModule.rst | |
parent | 92cac9621217157c5f1947ee0da4ade018cef75c (diff) | |
download | extra-cmake-modules-79caa00d0949138b237ecced794a903237bcf2b5.tar.gz extra-cmake-modules-79caa00d0949138b237ecced794a903237bcf2b5.tar.bz2 |
KDEGitCommitHooks: Check if KDEClangFormat was included
If the module is not included, the current message would be misleading.
Including the module by default is undesirable, because we print out a warning
that we do not override an existing module. While this warning is reasonable in most
cases, it is annoying for projects that have a custom clang-format file.
Checking if a .clang-format file exists and if not including KDEClangFormat is fragile,
because it depends on the include order of the modules.
Considering all that, a proper warning with instructions on how to fix the issue
is IMHO the best way to go.
Diffstat (limited to 'docs/module/ECMQmlModule.rst')
0 files changed, 0 insertions, 0 deletions