From 2187d7250fe4671d274e6e5948bd901969788e18 Mon Sep 17 00:00:00 2001 From: ChelSlava Date: Mon, 30 Mar 2026 10:21:10 +0300 Subject: [PATCH] Fix wildcard imports and resource leak --- src/robotide/application/__init__.py | 2 +- src/robotide/publish/__init__.py | 7 ++++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/src/robotide/application/__init__.py b/src/robotide/application/__init__.py index 7ecd71762..79a291173 100644 --- a/src/robotide/application/__init__.py +++ b/src/robotide/application/__init__.py @@ -14,6 +14,6 @@ # limitations under the License. from .application import RIDE -from .debugconsole import * +from . import debugconsole from ..controller import Project from .editorprovider import EditorProvider diff --git a/src/robotide/publish/__init__.py b/src/robotide/publish/__init__.py index f1462e14c..0eca86143 100644 --- a/src/robotide/publish/__init__.py +++ b/src/robotide/publish/__init__.py @@ -120,10 +120,11 @@ def important_action(self): import os -from .messages import * +from .messages import * # Wildcard import OK: messages.py defines __all__ from .publisher import PUBLISHER def get_html_message(name): - return open(os.path.join( - os.path.dirname(__file__), 'htmlmessages', '{}.html'.format(name))).read() + path = os.path.join(os.path.dirname(__file__), 'htmlmessages', f'{name}.html') + with open(path, encoding='utf-8') as f: + return f.read()