Apply some Pylint fixes

CHANGELOG.md

@@ -50,6 +50,7 @@ All notable changes to this project will be documented in this file.
 -   \[INTERNALS\] Remove `ast.basic.TypedAstNode._dtype`. The datatype can still be accessed as it is contained within the class type.
 -   \[INTERNALS\] Removed unused and undocumented function `get_function_from_ast`.
 -   \[INTERNALS\] Remove unused parameters `expr`, `status` and `like` from `pyccel.ast.core.Assign`.
+-   \[INTERNALS\] Remove unnecessary functions from `CodePrinter` : `_get_statement`, `_get_comment`.
 
 ## \[1.11.2\] - 2024-03-05
 

pyccel/ast/core.py

@@ -572,7 +572,6 @@ class CodeBlock(PyccelAstNode):
     __slots__ = ('_body','_unravelled')
     _attribute_nodes = ('_body',)
 
-
     def __init__(self, body, unravelled = False):
         ls = []
         for i in body:
@@ -716,15 +715,21 @@ def rhs(self):
 
 
 class SymbolicAssign(PyccelAstNode):
+    """
+    Represents symbolic aliasing for code generation.
 
-    """Represents symbolic aliasing for code generation. An alias is any statement of the
+    Represents symbolic aliasing for code generation. An alias is any statement of the
     form `lhs := rhs` where
 
+    This code needs investigating to find out if it is still useful.
+
     Parameters
     ----------
     lhs : PyccelSymbol
+        Why not a Variable? To be investigated.
 
     rhs : Range
+        Apparently a range.
 
     Examples
     --------
@@ -734,7 +739,6 @@ class SymbolicAssign(PyccelAstNode):
     >>> r = Range(0, 3)
     >>> y = PyccelSymbol('y')
     >>> SymbolicAssign(y, r)
-
     """
     __slots__ = ('_lhs', '_rhs')
     _attribute_nodes = ('_lhs', '_rhs')
@@ -745,7 +749,7 @@ def __init__(self, lhs, rhs):
         super().__init__()
 
     def __str__(self):
-        return '{0} := {1}'.format(str(self.lhs), str(self.rhs))
+        return f'{self.lhs} := {self.rhs}'
 
     @property
     def lhs(self):
@@ -3728,7 +3732,7 @@ def __init__(self, source, target = None, ignore_at_print = False, mod = None):
 
         self._source = source
         self._target = set()
-        self._source_mod      = mod
+        self._source_mod = mod
         self._ignore_at_print = ignore_at_print
         if target is None:
             if pyccel_stage == "syntactic":
@@ -3806,7 +3810,7 @@ def __str__(self):
         if len(self.target) == 0:
             return f'import {source}'
         else:
-            target = ', '.join([str(i) for i in self.target])
+            target = ', '.join(str(i) for i in self.target)
             return f'from {source} import {target}'
 
     def define_target(self, new_target):
@@ -4046,13 +4050,15 @@ class Raise(PyccelAstNode):
 
 
 class SymbolicPrint(PyccelAstNode):
+    """
+    Represents a print function of symbolic expressions in the code.
 
-    """Represents a print function of symbolic expressions in the code.
+    Represents a print function of symbolic expressions in the code.
 
     Parameters
     ----------
     expr : TypedAstNode
-        The expression to print
+        The expression to print.
 
     Examples
     --------
@@ -4072,7 +4078,7 @@ def __init__(self, expr):
         for i in expr:
             if not isinstance(i, (Lambda, SymbolicAssign,
                               SympyFunction)):
-                raise TypeError('Expecting Lambda, SymbolicAssign, SympyFunction for {}'.format(i))
+                raise TypeError(f'Expecting Lambda, SymbolicAssign, SympyFunction for {i}')
 
         self._expr = expr
 

pyccel/codegen/pipeline.py

@@ -145,7 +145,7 @@ def execute_pyccel(fname, *,
     # Unified way to handle errors: print formatted error message, then move
     # to original working directory. Caller should then raise exception.
     def handle_error(stage):
-        print('\nERROR at {} stage'.format(stage))
+        print(f'\nERROR at {stage} stage')
         errors.check()
         os.chdir(base_dirpath)
 
@@ -283,7 +283,7 @@ def handle_error(stage):
         output_file = (output_name + '.py') if output_name else os.path.basename(fname)
         new_location = os.path.join(folder, output_file)
         if verbose:
-            print("cp {} {}".format(fname, new_location))
+            print(f"cp {fname} {new_location}")
         shutil.copyfile(fname, new_location)
 
         # Change working directory back to starting point
@@ -432,7 +432,7 @@ def get_module_dependencies(parser, deps):
     shutil.move(generated_filepath, target)
     generated_filepath = target
     if verbose:
-        print( '> Shared library has been created: {}'.format(generated_filepath))
+        print( '> Shared library has been created:', generated_filepath)
 
     if codegen.is_program:
         generated_program_filename = os.path.basename(generated_program_filepath)
@@ -441,7 +441,7 @@ def get_module_dependencies(parser, deps):
         generated_program_filepath = target
 
         if verbose:
-            print( '> Executable has been created: {}'.format(generated_program_filepath))
+            print( '> Executable has been created:', generated_program_filepath)
 
     # Print all warnings now
     if errors.has_warnings():