Skip to content

Commit

Permalink
fix more ruff issues
Browse files Browse the repository at this point in the history
  • Loading branch information
barrust committed Dec 27, 2024
1 parent 869c627 commit 676a85e
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/python-package.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: ['3.8', '3.9', '3.10', '3.11', '3.12', '3.13']
python-version: ['3.9', '3.10', '3.11', '3.12', '3.13']

steps:
- uses: actions/checkout@v4
Expand Down
6 changes: 2 additions & 4 deletions probables/__init__.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
""" pyprobables module """

from typing import List
"""pyprobables module"""

from probables.blooms import (
BloomFilter,
Expand All @@ -26,7 +24,7 @@
__email__ = "barrust@gmail.com"
__license__ = "MIT"
__version__ = "0.6.1"
__credits__: List[str] = []
__credits__: list[str] = []
__url__ = "https://github.com/barrust/pyprobables"
__bugtrack_url__ = "https://github.com/barrust/pyprobables/issues"

Expand Down
5 changes: 1 addition & 4 deletions probables/blooms/expandingbloom.py
Original file line number Diff line number Diff line change
Expand Up @@ -146,10 +146,7 @@ def check_alt(self, hashes: HashResultsT) -> bool:
Returns:
bool: `True` if the element is likely present; `False` if definately not present"""
# return any(.check_alt(hashes))
for blm in self._blooms:
if blm.check_alt(hashes):
return True
return False
return any(blm.check_alt(hashes) for blm in self._blooms)

def add(self, key: KeyT, force: bool = False) -> None:
"""Add the key to the Bloom Filter
Expand Down

0 comments on commit 676a85e

Please sign in to comment.