Skip to content

Commit

Permalink
check_logs.py: Use longer initial build variable name
Browse files Browse the repository at this point in the history
Helps clear up the following pylint warnings:

  check_logs.py:52:4: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:93:15: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:101:14: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:109:14: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:124:23: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:130:23: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:135:23: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:184:21: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:210:13: W0621: Redefining name 'build' from outer scope (line 274) (redefined-outer-name)
  check_logs.py:249:14: W0621: Redefining name "build' from outer scope (line 274) (redefined-outer-name)

Signed-off-by: Nathan Chancellor <nathan@kernel.org>
  • Loading branch information
nathanchance committed Dec 7, 2022
1 parent 686a3b0 commit 8c68669
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions check_logs.py
Expand Up @@ -271,11 +271,11 @@ def boot_test(build):
print_red(f"${var} must be specified")
show_builds()
sys.exit(1)
build = verify_build()
verified_build = verify_build()
print_yellow("Register clang error/warning problem matchers")
for problem_matcher in glob.glob(".github/problem-matchers/*.json"):
print(f"::add-matcher::{problem_matcher}")
print_clang_info(build)
check_log(build)
check_built_config(build)
boot_test(build)
print_clang_info(verified_build)
check_log(verified_build)
check_built_config(verified_build)
boot_test(verified_build)

0 comments on commit 8c68669

Please sign in to comment.