scripts: west_commands: build_helpers: Fix linter issues

Fix issues reported by ruff.

Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>

Author: pdgendt

.ruff-excludes.toml

@@ -1082,11 +1082,6 @@
     "F541",     # https://docs.astral.sh/ruff/rules/f-string-missing-placeholders
     "I001",     # https://docs.astral.sh/ruff/rules/unsorted-imports
 ]
-"./scripts/west_commands/build_helpers.py" = [
-    "E402",     # https://docs.astral.sh/ruff/rules/module-import-not-at-top-of-file
-    "I001",     # https://docs.astral.sh/ruff/rules/unsorted-imports
-    "UP032",    # https://docs.astral.sh/ruff/rules/f-string
-]
 "./scripts/west_commands/completion.py" = [
     "UP015",    # https://docs.astral.sh/ruff/rules/redundant-open-modes
     "UP032",    # https://docs.astral.sh/ruff/rules/f-string

scripts/west_commands/build_helpers.py

@@ -10,10 +10,11 @@
 See build.py for the build command itself.
 '''
 
-import zcmake
 import os
 import sys
 from pathlib import Path
+
+import zcmake
 from west import log
 from west.configuration import config
 from west.util import escapes_directory
@@ -22,19 +23,19 @@
 # twister also uses the implementation
 script_dir = os.path.dirname(os.path.dirname(os.path.realpath(__file__)))
 sys.path.insert(0, os.path.join(script_dir, "pylib/build_helpers/"))
-from domains import Domains
+from domains import Domains  # noqa: E402
 
 DEFAULT_BUILD_DIR = 'build'
 '''Name of the default Zephyr build directory.'''
 
 DEFAULT_CMAKE_GENERATOR = 'Ninja'
 '''Name of the default CMake generator.'''
 
-FIND_BUILD_DIR_DESCRIPTION = '''\
-If the build directory is not given, the default is {}/ unless the
+FIND_BUILD_DIR_DESCRIPTION = f'''\
+If the build directory is not given, the default is {DEFAULT_BUILD_DIR}/ unless the
 build.dir-fmt configuration variable is set. The current directory is
 checked after that. If either is a Zephyr build directory, it is used.
-'''.format(DEFAULT_BUILD_DIR)
+'''
 
 def _resolve_build_dir(fmt, guess, cwd, **kwargs):
     # Remove any None values, we do not want 'None' as a string
@@ -100,14 +101,14 @@ def find_build_dir(dir, guess=False, **kwargs):
         cwd = os.getcwd()
         default = config.get('build', 'dir-fmt', fallback=DEFAULT_BUILD_DIR)
         default = _resolve_build_dir(default, guess, cwd, **kwargs)
-        log.dbg('config dir-fmt: {}'.format(default), level=log.VERBOSE_EXTREME)
+        log.dbg(f'config dir-fmt: {default}', level=log.VERBOSE_EXTREME)
         if default and is_zephyr_build(default):
             build_dir = default
         elif is_zephyr_build(cwd):
             build_dir = cwd
         else:
             build_dir = default
-    log.dbg('build dir: {}'.format(build_dir), level=log.VERBOSE_EXTREME)
+    log.dbg(f'build dir: {build_dir}', level=log.VERBOSE_EXTREME)
     if build_dir:
         return os.path.abspath(build_dir)
     else: