fixed some bugs

Author: manfred-kaiser

sifter/commands/require.py

@@ -20,7 +20,7 @@ class CommandRequire(Command):
 
     def evaluate(self, message: Message, state: EvaluationState) -> Optional[Actions]:
         ext_name_list = self.positional_args[0]
-        for ext_name in ext_name_list:
+        for ext_name in ext_name_list:  # type: ignore
             if not sifter.handler.get('extension', ext_name):
                 raise RuntimeError(
                     "Required extension '%s' not supported"

sifter/grammar/grammar.py

@@ -5,6 +5,7 @@
 from sifter.grammar.test import Test
 from sifter.grammar.command import Command
 from typing import (
+    Any,
     TYPE_CHECKING,
     cast,
     Text
@@ -32,7 +33,7 @@ def __init__(self) -> None:
         self.parser = self.make_parser(self)
 
     @staticmethod
-    def make_parser(mod) -> 'LRParser':
+    def make_parser(mod: Any) -> 'LRParser':
         return ply.yacc.yacc(
             module=mod
         )

sifter/grammar/lexer.py

@@ -12,7 +12,7 @@
 import ply.lex  # type: ignore
 
 if TYPE_CHECKING:
-    from ply.lex import LexToken  # type: ignore
+    from ply.lex import LexToken
 
 
 class SieveLexer():

sifter/grammar/rule.py

@@ -33,7 +33,7 @@ class Rule(object):
     POSITIONAL_ARGS: Optional[List[Validator]] = None
 
     HAS_TESTS: bool = True
-    TESTS_MIN: Optional[int] = 0
+    TESTS_MIN: int = 0
     TESTS_MAX: Optional[int] = None
 
     @classmethod

sifter/tests/exists.py

@@ -19,7 +19,7 @@ def evaluate(self, message: Message, state: EvaluationState) -> Optional[bool]:
         headers = self.positional_args[0]
         if not isinstance(headers, list):
             raise ValueError("TestExists.headers must be a list")
-        for header in self.headers:
+        for header in headers:
             if header not in message:
                 return False
         return True