diff options
| author | Patrik Teivonen <patrik.teivonen@qt.io> | 2022-06-16 12:07:05 +0300 |
|---|---|---|
| committer | Patrik Teivonen <patrik.teivonen@qt.io> | 2022-07-11 07:43:46 +0000 |
| commit | 45312f31b5439e99598a553d0fffd7c7060bfbaf (patch) | |
| tree | 88207d30e9ac6f4195d9ec3f4d54c6124454b5e8 /packaging-tools/build_clang_qdoc.py | |
| parent | c444074df00e88910bd1b0f8e257a1307419f715 (diff) | |
Add pre-commit hook for flake8, PEP8 compliant comments
Change block and inline comments to be PEP8 compliant.
(Note: line lengths will be enforced in a separate patch)
Add pre-commit hook for flake8, and enable only checks relevant
to this patch:
- E265 Block comment should start with '# '
- E266 Too many leading '#' for block comment
- E261 At least two spaces before inline comment
- E262 Inline comment should start with '# '
- E115 expected an indented block (comment)
Change-Id: I0218772bfbea514b4a0d8ecc461845c4563396da
Reviewed-by: Akseli Salovaara <akseli.salovaara@qt.io>
Reviewed-by: Iikka Eklund <iikka.eklund@qt.io>
Diffstat (limited to 'packaging-tools/build_clang_qdoc.py')
| -rw-r--r-- | packaging-tools/build_clang_qdoc.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/packaging-tools/build_clang_qdoc.py b/packaging-tools/build_clang_qdoc.py index 32fa9ad32..f3a636fea 100644 --- a/packaging-tools/build_clang_qdoc.py +++ b/packaging-tools/build_clang_qdoc.py @@ -116,7 +116,7 @@ def build_environment(toolchain, bitness): environment['CXX'] = 'cl' return environment else: - return None # == process environment + return None # == process environment def is_msvc_toolchain(toolchain): return 'msvc' in toolchain @@ -206,7 +206,7 @@ def build_clang(toolchain, src_path, build_path, install_path, bitness=64, envir do_execute_sub_process(cmake_cmd, build_path, extra_env=environment) build_targets = ['install/strip'] if is_msvc_toolchain(toolchain): - build_targets = ['install'] # There is no 'install/strip' for nmake. + build_targets = ['install'] # There is no 'install/strip' for nmake. build_and_install(toolchain, build_path, environment, ['libclang', 'clang', 'llvm-config'], build_targets) def check_clang(toolchain, build_path, environment): |