cleanup

joeyagreco · Sep 21, 2024 · a4ca00d · a4ca00d
1 parent e02bc77
commit a4ca00d
Show file tree

Hide file tree

Showing 5 changed files with 116 additions and 107 deletions.
diff --git a/sleeper/api/_utils.py b/sleeper/api/_utils.py
@@ -1,4 +1,4 @@
-from typing import Optional
+from typing import Any, Optional
 
 import requests
 
@@ -13,7 +13,7 @@ def build_route(base_url: str, version: Optional[str], *args) -> str:
         return f"{base_url}/{'/'.join(args)}"
 
 
-def get(url: str) -> any:
+def get(url: str) -> Any:
     response = requests.get(url)
     response.raise_for_status()
     return response.json()
@@ -25,7 +25,7 @@ def get_content(url: str) -> bytes:
     return response.content
 
 
-def add_filters(url: str, *args: list[tuple[str, any]]) -> str:
+def add_filters(url: str, *args: tuple[str, Any]) -> str:
     """
     Adds filters to the given url.
     """

diff --git a/sleeper/api/draft.py b/sleeper/api/draft.py
@@ -13,51 +13,54 @@
 
 
 def get_user_drafts_for_year(*, user_id: str, sport: Sport, year: int) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        USER_ROUTE,
-        user_id,
-        DRAFTS_ROUTE,
-        sport,
-        year,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            USER_ROUTE,
+            user_id,
+            DRAFTS_ROUTE,
+            sport,
+            year,
+        )
     )
-    return get(url)
 
 
 def get_drafts_in_league(*, league_id: str) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        LEAGUE_ROUTE,
-        league_id,
-        DRAFTS_ROUTE,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            LEAGUE_ROUTE,
+            league_id,
+            DRAFTS_ROUTE,
+        )
     )
-    return get(url)
 
 
 def get_draft(*, draft_id: str) -> dict:
-    url = build_route(SLEEPER_APP_BASE_URL, VERSION, DRAFT_ROUTE, draft_id)
-    return get(url)
+    return get(build_route(SLEEPER_APP_BASE_URL, VERSION, DRAFT_ROUTE, draft_id))
 
 
 def get_player_draft_picks(*, draft_id: str) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        DRAFT_ROUTE,
-        draft_id,
-        PICKS_ROUTE,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            DRAFT_ROUTE,
+            draft_id,
+            PICKS_ROUTE,
+        )
     )
-    return get(url)
 
 
 def get_traded_draft_picks(*, draft_id: str) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        DRAFT_ROUTE,
-        draft_id,
-        TRADED_PICKS_ROUTE,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            DRAFT_ROUTE,
+            draft_id,
+            TRADED_PICKS_ROUTE,
+        )
     )
-    return get(url)
diff --git a/sleeper/api/league.py b/sleeper/api/league.py
@@ -18,102 +18,108 @@
 
 
 def get_league(*, league_id: str) -> dict:
-    url = build_route(SLEEPER_APP_BASE_URL, VERSION, LEAGUE_ROUTE, league_id)
-    return get(url)
+    return get(build_route(SLEEPER_APP_BASE_URL, VERSION, LEAGUE_ROUTE, league_id))
 
 
 def get_user_leagues_for_year(*, user_id: str, sport: Sport, year: int) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        USER_ROUTE,
-        user_id,
-        LEAGUES_ROUTE,
-        sport,
-        year,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            USER_ROUTE,
+            user_id,
+            LEAGUES_ROUTE,
+            sport,
+            year,
+        )
     )
-    return get(url)
 
 
 def get_rosters(*, league_id: str) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        LEAGUE_ROUTE,
-        league_id,
-        ROSTERS_ROUTE,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            LEAGUE_ROUTE,
+            league_id,
+            ROSTERS_ROUTE,
+        )
     )
-    return get(url)
 
 
 def get_users_in_league(*, league_id: str) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        LEAGUE_ROUTE,
-        league_id,
-        USERS_ROUTE,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            LEAGUE_ROUTE,
+            league_id,
+            USERS_ROUTE,
+        )
     )
-    return get(url)
 
 
 def get_matchups_for_week(*, league_id: str, week: int) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        LEAGUE_ROUTE,
-        league_id,
-        MATCHUPS_ROUTE,
-        week,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            LEAGUE_ROUTE,
+            league_id,
+            MATCHUPS_ROUTE,
+            week,
+        )
     )
-    return get(url)
 
 
 def get_winners_bracket(*, league_id: str) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        LEAGUE_ROUTE,
-        league_id,
-        WINNERS_BRACKET_ROUTE,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            LEAGUE_ROUTE,
+            league_id,
+            WINNERS_BRACKET_ROUTE,
+        )
     )
-    return get(url)
 
 
 def get_losers_bracket(*, league_id: str) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        LEAGUE_ROUTE,
-        league_id,
-        LOSERS_BRACKET_ROUTE,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            LEAGUE_ROUTE,
+            league_id,
+            LOSERS_BRACKET_ROUTE,
+        )
     )
-    return get(url)
 
 
 def get_transactions(*, league_id: str, week: int) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        LEAGUE_ROUTE,
-        league_id,
-        TRANSACTIONS_ROUTE,
-        week,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            LEAGUE_ROUTE,
+            league_id,
+            TRANSACTIONS_ROUTE,
+            week,
+        )
     )
-    return get(url)
 
 
 def get_traded_picks(*, league_id: str) -> list[dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        LEAGUE_ROUTE,
-        league_id,
-        TRADED_PICKS_ROUTE,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            LEAGUE_ROUTE,
+            league_id,
+            TRADED_PICKS_ROUTE,
+        )
     )
-    return get(url)
 
 
 def get_sport_state(sport: Sport) -> dict:
-    url = build_route(SLEEPER_APP_BASE_URL, VERSION, STATE_ROUTE, sport)
-    return get(url)
+    return get(build_route(SLEEPER_APP_BASE_URL, VERSION, STATE_ROUTE, sport))
diff --git a/sleeper/api/player.py b/sleeper/api/player.py
@@ -11,13 +11,14 @@
 
 
 def get_all_players(*, sport: Sport) -> dict[str, dict]:
-    url = build_route(
-        SLEEPER_APP_BASE_URL,
-        VERSION,
-        PLAYERS_ROUTE,
-        sport,
+    return get(
+        build_route(
+            SLEEPER_APP_BASE_URL,
+            VERSION,
+            PLAYERS_ROUTE,
+            sport,
+        )
     )
-    return get(url)
 
 
 def get_trending_players(

diff --git a/sleeper/api/user.py b/sleeper/api/user.py
@@ -4,5 +4,4 @@
 
 def get_user(*, identifier: str) -> dict:
     # identifier can be username or user id
-    url = build_route(SLEEPER_APP_BASE_URL, VERSION, USER_ROUTE, f"{identifier}")
-    return get(url)
+    return get(build_route(SLEEPER_APP_BASE_URL, VERSION, USER_ROUTE, f"{identifier}"))