Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix issues recollection url #2840

Merged
merged 1 commit into from
Jun 30, 2024
Merged

Fix issues recollection url #2840

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion augur/tasks/github/issues/tasks.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import logging

Check warning on line 1 in augur/tasks/github/issues/tasks.py

View workflow job for this annotation

GitHub Actions / runner / pylint 

[pylint] reported by reviewdog 🐶
C0114: Missing module docstring (missing-module-docstring)

Raw Output:
augur/tasks/github/issues/tasks.py:1:0: C0114: Missing module docstring (missing-module-docstring)
import traceback
from datetime import timedelta, timezone

Expand All @@ -7,7 +7,7 @@

from augur.tasks.init.celery_app import celery_app as celery
from augur.tasks.init.celery_app import AugurCoreRepoCollectionTask
from augur.application.db.data_parse import *

Check warning on line 10 in augur/tasks/github/issues/tasks.py

View workflow job for this annotation

GitHub Actions / runner / pylint 

[pylint] reported by reviewdog 🐶
W0401: Wildcard import augur.application.db.data_parse (wildcard-import)

Raw Output:
augur/tasks/github/issues/tasks.py:10:0: W0401: Wildcard import augur.application.db.data_parse (wildcard-import)
from augur.tasks.github.util.github_data_access import GithubDataAccess
from augur.tasks.github.util.github_random_key_auth import GithubRandomKeyAuth
from augur.tasks.github.util.util import add_key_value_pair_to_dicts, get_owner_repo
Expand Down Expand Up @@ -65,7 +65,7 @@
url = f"https://api.github.com/repos/{owner}/{repo}/issues?state=all"

if since:
url += since.isoformat()
url += f"&since={since.isoformat()}"

github_data_access = GithubDataAccess(key_auth, logger)

Expand All @@ -76,7 +76,7 @@

return list(issues_paginator)

def process_issues(issues, task_name, repo_id, logger) -> None:

Check warning on line 79 in augur/tasks/github/issues/tasks.py

View workflow job for this annotation

GitHub Actions / runner / pylint 

[pylint] reported by reviewdog 🐶
R0914: Too many local variables (32/30) (too-many-locals)

Raw Output:
augur/tasks/github/issues/tasks.py:79:0: R0914: Too many local variables (32/30) (too-many-locals)

# get repo_id or have it passed
tool_source = "Issue Task"
Expand All @@ -87,7 +87,7 @@
issue_mapping_data = {}
issue_total = len(issues)
contributors = []
for index, issue in enumerate(issues):

Check warning on line 90 in augur/tasks/github/issues/tasks.py

View workflow job for this annotation

GitHub Actions / runner / pylint 

[pylint] reported by reviewdog 🐶
W0612: Unused variable 'index' (unused-variable)

Raw Output:
augur/tasks/github/issues/tasks.py:90:8: W0612: Unused variable 'index' (unused-variable)

# calls is_valid_pr_block to see if the data is a pr.
# if it is a pr we skip it because we don't need prs
Expand Down Expand Up @@ -163,7 +163,7 @@


# add the issue id to the lables and assignees, then add them to a list of dicts that will be inserted soon
dict_key = "issue_id"

Check warning on line 166 in augur/tasks/github/issues/tasks.py

View workflow job for this annotation

GitHub Actions / runner / pylint 

[pylint] reported by reviewdog 🐶
W0612: Unused variable 'dict_key' (unused-variable)

Raw Output:
augur/tasks/github/issues/tasks.py:166:8: W0612: Unused variable 'dict_key' (unused-variable)
issue_label_dicts += add_key_value_pair_to_dicts(other_issue_data["labels"], "issue_id", issue_id)
issue_assignee_dicts += add_key_value_pair_to_dicts(other_issue_data["assignees"], "issue_id", issue_id)

Expand Down
Loading