Fixed new pylint rule violations.

Author: LarsAsplund

tests/lint/pylintrc

@@ -36,8 +36,8 @@ load-plugins=
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
 #disable=
-disable=too-few-public-methods, locally-disabled, redefined-variable-type, duplicate-code, file-ignored,
-  useless-object-inheritance, bad-continuation,
+disable=too-few-public-methods, locally-disabled, duplicate-code, file-ignored,
+  useless-object-inheritance,
 # Failed to suppress this locally in ostools.py
   subprocess-popen-preexec-fn
 
@@ -48,11 +48,6 @@ disable=too-few-public-methods, locally-disabled, redefined-variable-type, dupli
 # mypackage.mymodule.MyReporterClass.
 output-format=text
 
-# Put messages in a separate file for each module / package specified on the
-# command line instead of printing them on stdout. Reports (if any) will be
-# written in a file name "pylint_global.[txt|html]".
-files-output=no
-
 # Tells whether to display a full report or only the messages
 reports=no
 
@@ -84,9 +79,6 @@ additional-builtins=
 
 [BASIC]
 
-# List of builtins function names that should not be used, separated by a comma
-bad-functions=map,filter,apply,input
-
 # Regular expression which should only match correct module names
 module-rgx=(([a-z_][a-z0-9_]*)|([A-Z][a-zA-Z0-9]+))$
 
@@ -163,9 +155,6 @@ ignore-long-lines=^\s*(# )?<?https?://\S+>?$
 # else.
 single-line-if-stmt=no
 
-# List of optional constructs for which whitespace checking is disabled
-no-space-check=trailing-comma,dict-separator
-
 # Maximum number of lines in a module
 max-module-lines=1000
 

vunit/sim_if/__init__.py

@@ -177,7 +177,7 @@ def supports_coverage():
         """
         return False
 
-    def merge_coverage(self, file_name, args):  # pylint: disable=unused-argument, no-self-use
+    def merge_coverage(self, file_name, args):  # pylint: disable=unused-argument
         """
         Hook for simulator interface to creating coverage reports
         """

vunit/sim_if/ghdl.py

@@ -250,7 +250,7 @@ def compile_vhdl_file_command(self, source_file):
         flags = source_file.compile_options.get("ghdl.flags", [])
         if flags != []:
             warn(
-                ("'ghdl.flags' is deprecated and it will be removed in future releases; " "use 'ghdl.a_flags' instead"),
+                ("'ghdl.flags' is deprecated and it will be removed in future releases; use 'ghdl.a_flags' instead"),
                 Warning,
             )
             a_flags += flags

vunit/test/bench_list.py

@@ -111,7 +111,7 @@ def tb_filter(design_unit):
 
     elif has_runner_cfg and not has_tb_name:
         LOGGER.warning(
-            "%s %s has runner_cfg %s but the file name and the %s name does not match regex %s\n" "in file %s",
+            "%s %s has runner_cfg %s but the file name and the %s name does not match regex %s\nin file %s",
             design_unit_type,
             design_unit.name,
             generic_type,