diff --git a/nemoguardrails/utils.py b/nemoguardrails/utils.py index 1afef6e4..8ed0ac3c 100644 --- a/nemoguardrails/utils.py +++ b/nemoguardrails/utils.py @@ -341,7 +341,7 @@ def get_railsignore_path(path: Optional[str] = None) -> Optional[Path]: return None -def get_railsignore_patterns(railsignore_path) -> Set[str]: +def get_railsignore_patterns(railsignore_path: Path) -> Set[str]: """ Helper to retrieve all specified patterns in railsignore. Returns: @@ -352,8 +352,6 @@ def get_railsignore_patterns(railsignore_path) -> Set[str]: if railsignore_path is None: return ignored_patterns - # railsignore_path = get_railsignore_path() - # File doesn't exist or is empty if not railsignore_path.exists() or not os.path.getsize(railsignore_path): return ignored_patterns @@ -377,13 +375,13 @@ def get_railsignore_patterns(railsignore_path) -> Set[str]: return ignored_patterns -def is_ignored_by_railsignore(filename, ignore_patterns) -> bool: +def is_ignored_by_railsignore(filename: str, ignore_patterns: str) -> bool: + """Helper to verify if a filename should be ignored by a railsignore pattern""" ignore = False - # Load candidate patterns from railsignore - for pattern in ignore_patterns: if fnmatch.fnmatch(filename, pattern): ignore = True + break return ignore