diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 6a6ba2af..8364d97b 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -1,13 +1,13 @@ repos: - repo: https://github.com/psf/black - rev: 22.8.0 + rev: 23.1.0 hooks: - id: black - repo: https://github.com/PyCQA/isort - rev: 5.10.1 + rev: 5.12.0 hooks: - id: isort - repo: https://github.com/PyCQA/flake8 - rev: 5.0.4 + rev: 6.0.0 hooks: - id: flake8 diff --git a/jazzband/members/tasks.py b/jazzband/members/tasks.py index e5e82eaa..eb78cfdb 100644 --- a/jazzband/members/tasks.py +++ b/jazzband/members/tasks.py @@ -17,7 +17,6 @@ def sync_members(): # use a lock to make sure we don't run this multiple times with redis.lock("sync_members", ttl=ONE_MINUTE * 14): - members_data = github.get_members() User.sync(members_data) diff --git a/jazzband/projects/tasks.py b/jazzband/projects/tasks.py index 472532db..37d583a5 100644 --- a/jazzband/projects/tasks.py +++ b/jazzband/projects/tasks.py @@ -46,7 +46,6 @@ def update_project_by_hook(hook_id): # use a lock to make sure we don't run this multiple times with redis.lock(f"project-update-by-hook-{project_name}", ttl=ONE_MINUTE): - # if there already was an issue created, just stop here if not project.transfer_issue_url: # get list of roadies and set them as the default assignees @@ -90,7 +89,6 @@ def send_new_upload_notifications(project_id=None): recipients = set() for lead_member in lead_members + list(User.roadies()): - primary_email = lead_member.email_addresses.filter( EmailAddress.primary.is_(True), EmailAddress.verified.is_(True) ).first() @@ -150,7 +148,6 @@ def sync_project_members(): in GitHub anymore. """ with redis.lock("sync_project_members", ttl=ONE_MINUTE * 14): - teams = github.get_teams() for team in teams: