Skip to content

Commit 0b958c4

Browse files
Apply suggestions from code review
Co-authored-by: Kathy Pippert <[email protected]>
1 parent 141bf22 commit 0b958c4

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

pyaedt/misc/Run_Toolkit_Manager.py_build

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ def main():
7373

7474
def check_file(file_path):
7575
if not os.path.isfile(file_path):
76-
show_error('"{}" does not exist. Please click on the "Install PyAEDT" button in the Automation ribbon.'.format(
76+
show_error('"{}" does not exist. Click the "Install PyAEDT" button in the Automation ribbon.'.format(
7777
file_path))
7878

7979

pyaedt/misc/aedtlib_personalib_install.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ def install_toolkit(toolkit_dir, product, aedt_version, is_student_version=False
149149
os.makedirs(lib_dir, exist_ok=True)
150150
os.makedirs(tool_dir, exist_ok=True)
151151
files_to_copy = ["Console", "Run_PyAEDT_Script", "Jupyter", "Run_Toolkit_Manager"]
152-
# Remove hard-coded version number from Python virtual environment path, and replace it with the corresponding AEDT
152+
# Remove hard-coded version number from Python virtual environment path and replace it with the corresponding AEDT
153153
# version's Python virtual environment.
154154
version_agnostic = False
155155
if aedt_version[2:6].replace(".", "") in sys.executable:

pyaedt/misc/toolkit_manager.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ def is_toolkit_installed(toolkit_name):
114114

115115

116116
def open_window(window, window_name, open_source_toolkits):
117-
"""Open window action"""
117+
"""Open a window."""
118118
if not hasattr(window, "opened"):
119119
window.opened = True
120120
window.title(window_name)
@@ -217,9 +217,9 @@ def button_is_clicked(
217217
executable_interpreter=executable_interpreter,
218218
)
219219
else:
220-
desktop.logger.info("PyAEDT environment is not installed")
220+
desktop.logger.info("PyAEDT environment is not installed.")
221221
else:
222-
desktop.logger.info("Uninstall {}".format(name))
222+
desktop.logger.info("Uninstall {}.".format(name))
223223
desktop.remove_script_from_menu(name, product=toolkit_level)
224224

225225
desktop.odesktop.CloseAllWindows()

0 commit comments

Comments
 (0)