Skip to content

Commit a5b5bf1

Browse files
fix: null safety
1 parent 90d84b5 commit a5b5bf1

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

codeflash/code_utils/env_utils.py

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -75,13 +75,14 @@ def ensure_codeflash_api_key() -> bool:
7575
@lru_cache(maxsize=1)
7676
def get_pr_number() -> Optional[int]:
7777
event_data = get_cached_gh_event_data()
78-
gh_pr_number = event_data["number"]
79-
if gh_pr_number is not None:
80-
return int(gh_pr_number)
78+
pr_number = event_data.get("number")
79+
if pr_number:
80+
return int(pr_number)
81+
8182
pr_number = os.environ.get("CODEFLASH_PR_NUMBER")
82-
if not pr_number:
83-
return None
84-
return int(pr_number)
83+
if pr_number:
84+
return int(pr_number)
85+
return None
8586

8687

8788
def ensure_pr_number() -> bool:
@@ -110,9 +111,7 @@ def get_cached_gh_event_data() -> dict[str, Any]:
110111

111112
def is_repo_a_fork() -> bool:
112113
event = get_cached_gh_event_data()
113-
if event is None:
114-
return False
115-
return bool(event["repository"]["fork"])
114+
return bool(event.get("repository", {}).get("fork", False))
116115

117116

118117
@lru_cache(maxsize=1)

0 commit comments

Comments
 (0)