From 2f58d306c5a757faa3f084aeddf5d4035073c50c Mon Sep 17 00:00:00 2001 From: lbrealdev Date: Sun, 19 Mar 2023 20:37:12 +0100 Subject: [PATCH 1/2] refactor: list-repository function --- github-rest-api.py | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/github-rest-api.py b/github-rest-api.py index 204f009..7ef3788 100644 --- a/github-rest-api.py +++ b/github-rest-api.py @@ -171,11 +171,14 @@ def list_repositories(limit: int, property: str, role: str) -> None: f"{GITHUB_URL}/user/repos", headers=headers, params=params ) if resp.status_code == 200: - repos = json.loads(resp.text) - repo_names = [repo["full_name"] for repo in repos] - for repo_name in repo_names: - rich_output(f"- {repo_name}", fmt="blink bold green") - rich_output(f"\nTotal repositories: {len(repo_names)}", fmt="blink bold green") + repositories = json.loads(resp.text) + full_name = [repo["full_name"] for repo in repositories] + for repos in full_name: + rich_output(f"- {repos}", fmt="blink bold green") + rich_output( + f"\nTotal repositories: {len(full_name)}", + fmt="blink bold green", + ) else: rich_output( f"Failed to list repositories for {GITHUB_USER}\n" + From 0f93b3a0c1efa4ed69c6a87f5e81beb19d87c210 Mon Sep 17 00:00:00 2001 From: lbrealdev Date: Sun, 19 Mar 2023 22:40:41 +0100 Subject: [PATCH 2/2] fix: function type hints --- github-rest-api.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/github-rest-api.py b/github-rest-api.py index 7ef3788..adc09cf 100644 --- a/github-rest-api.py +++ b/github-rest-api.py @@ -21,13 +21,13 @@ } -def rich_output(input: str, fmt: str) -> None: +def rich_output(input: str, fmt: str): text = Text(input) text.stylize(fmt) console.print(text) -def get_repository(name: str, org: str) -> None: +def get_repository(name: str, org: str): if org is None: get_user_repository_info = requests.get( f"{GITHUB_URL}/repos/{GITHUB_USER}/{name}", headers=headers @@ -127,7 +127,7 @@ def create_repository(name: str, private: str, org: str): ) -def delete_repository(name: str, org: str) -> None: +def delete_repository(name: str, org: str): if org is not None: resp_org = requests.delete( f"{GITHUB_URL}/repos/{org}/{name}", headers=headers @@ -165,7 +165,7 @@ def delete_repository(name: str, org: str) -> None: ) -def list_repositories(limit: int, property: str, role: str) -> None: +def list_repositories(limit: int, property: str, role: str): params = {"per_page": limit, "sort": property, "type": role} resp = requests.get( f"{GITHUB_URL}/user/repos", headers=headers, params=params @@ -187,7 +187,7 @@ def list_repositories(limit: int, property: str, role: str) -> None: ) -def dependabot_security(name: str, option: str, org: str) -> None: +def dependabot_security(name: str, option: str, org: str): if org is not None: if option == "true": dependabot_on = requests.put(