[PERF] Checkers only run when enabled OCA#372

antonag32 · Jun 22, 2022 · 0332ea8 · 0332ea8
1 parent 1c91fce
commit 0332ea8
Show file tree

Hide file tree

Showing 3 changed files with 423 additions and 337 deletions.
diff --git a/pylint_odoo/checkers/format.py b/pylint_odoo/checkers/format.py
@@ -1,4 +1,3 @@
-
 import os
 import tokenize
 from sys import platform
@@ -48,7 +47,6 @@
 
 
 class FormatChecker(PylintOdooTokenChecker):
-
     name = settings.CFG_SECTION
     msgs = ODOO_MSGS
     odoo_check_versions = {
@@ -67,7 +65,7 @@ def get_magic_comment_type(self, comment, line_num):
                 return MAGIC_COMMENT_INTERPRETER
             elif "# -*- coding: " in comment or "# coding: " in comment:
                 if "# -*- coding: utf-8 -*-" in comment \
-                   or "# coding: utf-8" in comment:
+                        or "# coding: utf-8" in comment:
                     return MAGIC_COMMENT_CODING_UTF8
                 return MAGIC_COMMENT_CODING
             elif "# -*- encoding: " in comment:
@@ -90,19 +88,29 @@ def process_tokens(self, tokens):
                 if magic_comment_type != NO_IDENTIFIED:
                     tokens_identified[magic_comment_type] = [
                         token_content, line_num]
-                elif self.is_vim_comment(token_content):
+                elif (
+                    self.linter.is_message_enabled('use-vim-comment') and
+                    self.is_vim_comment(token_content)
+                ):
                     self.add_message('use-vim-comment', line=line_num)
-        if not tokens_identified.get(MAGIC_COMMENT_CODING_UTF8) and \
-           not os.path.basename(self.linter.current_file) == '__init__.py':
-            self.add_message('no-utf8-coding-comment', line=1)
-        if (tokens_identified.get(MAGIC_COMMENT_CODING_UTF8)):
-            self.add_message('unnecessary-utf8-coding-comment', line=1)
-        access_x = os.access(self.linter.current_file, os.X_OK)
-        interpreter_content, line_num = tokens_identified.get(
-            MAGIC_COMMENT_INTERPRETER, ['', 0])
-        if (not platform.startswith('win') and
-                bool(interpreter_content) != access_x):
-            self.add_message(
-                'incoherent-interpreter-exec-perm',
-                line=line_num, args=(
-                    os.path.basename(self.linter.current_file)))
+
+        if self.linter.is_message_enabled('no-utf8-coding-comment'):
+            if not tokens_identified.get(MAGIC_COMMENT_CODING_UTF8) and \
+                    not os.path.basename(self.linter.current_file) == '__init__.py':
+                self.add_message('no-utf8-coding-comment', line=1)
+
+        if self.linter.is_message_enabled('unnecessary-utf8-coding-comment'):
+            if tokens_identified.get(MAGIC_COMMENT_CODING_UTF8):
+                self.add_message('unnecessary-utf8-coding-comment', line=1)
+
+        if self.linter.is_message_enabled('incoherent-interpreter-exec-perm'):
+            access_x = os.access(self.linter.current_file, os.X_OK)
+            interpreter_content, line_num = tokens_identified.get(
+                MAGIC_COMMENT_INTERPRETER, ['', 0]
+            )
+            if not platform.startswith('win') and bool(interpreter_content) != access_x:
+                self.add_message(
+                    'incoherent-interpreter-exec-perm',
+                    line=line_num,
+                    args=(os.path.basename(self.linter.current_file))
+                )