From 4c484ad7a68700aa593ddccf6b698ab65bd7b4a9 Mon Sep 17 00:00:00 2001 From: aldbr Date: Fri, 9 Oct 2026 08:42:11 +0200 Subject: [PATCH] fix: annotate DIRAC.exit as NoReturn astroid 4.3.4 makes pylint report possibly-used-before-assignment in dirac_admin_add_site.py, because a branch that ends in DIRACExit(-1) is not known to terminate. Annotating DIRAC.exit with NoReturn lets pylint treat every DIRACExit call as the end of its branch. Co-Authored-By: Claude Opus 5.5 --- src/DIRAC/__init__.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/DIRAC/__init__.py b/src/DIRAC/__init__.py index a408647cc85..343857dbf8f 100755 --- a/src/DIRAC/__init__.py +++ b/src/DIRAC/__init__.py @@ -61,7 +61,7 @@ import sys from collections.abc import Sequence from pkgutil import extend_path -from typing import Any, Optional, Union +from typing import Any, NoReturn, Optional, Union __path__ = extend_path(__path__, __name__) @@ -293,7 +293,7 @@ def siteName(): from DIRAC.Core.Utilities.Platform import getPlatformString, getPlatform, getPlatformTuple -def exit(exitCode=0): +def exit(exitCode=0) -> NoReturn: """ Finish execution using callbacks """