wony617 commited on
Commit
b7c7d1c
·
1 Parent(s): b05265a

chore(log): require single LOG_REPO env (drop owner/name split)

Browse files

Simplify config by using only LOG_REPO in 'owner/repo' form. Update GitHubLogger to enforce this.

Files changed (1) hide show
  1. logger/github_logger.py +4 -8
logger/github_logger.py CHANGED
@@ -13,7 +13,7 @@ class GitHubLogger:
13
 
14
  Env vars:
15
  - LOG_GITHUB_TOKEN (fallback: GITHUB_TOKEN)
16
- - LOG_REPO or LOG_REPO_OWNER + LOG_REPO_NAME
17
  - LOG_BRANCH (default: 'log_event')
18
  - LOG_FILE_PATH (default: 'pr_success.log')
19
  """
@@ -27,13 +27,9 @@ class GitHubLogger:
27
  self._client = Github(token)
28
 
29
  repo_spec = os.environ.get("LOG_REPO")
30
- if repo_spec and "/" in repo_spec:
31
- self.owner, self.repo_name = repo_spec.split("/", 1)
32
- else:
33
- self.owner = os.environ.get("LOG_REPO_OWNER")
34
- self.repo_name = os.environ.get("LOG_REPO_NAME")
35
- if not self.owner or not self.repo_name:
36
- raise ValueError("Missing LOG_REPO or LOG_REPO_OWNER/LOG_REPO_NAME")
37
 
38
  self.branch = os.environ.get("LOG_BRANCH", "log_event")
39
  self.path = os.environ.get("LOG_FILE_PATH", "pr_success.log")
 
13
 
14
  Env vars:
15
  - LOG_GITHUB_TOKEN (fallback: GITHUB_TOKEN)
16
+ - LOG_REPO (format: owner/repo)
17
  - LOG_BRANCH (default: 'log_event')
18
  - LOG_FILE_PATH (default: 'pr_success.log')
19
  """
 
27
  self._client = Github(token)
28
 
29
  repo_spec = os.environ.get("LOG_REPO")
30
+ if not repo_spec or "/" not in repo_spec:
31
+ raise ValueError("Missing or invalid LOG_REPO. Expected 'owner/repo'.")
32
+ self.owner, self.repo_name = repo_spec.split("/", 1)
 
 
 
 
33
 
34
  self.branch = os.environ.get("LOG_BRANCH", "log_event")
35
  self.path = os.environ.get("LOG_FILE_PATH", "pr_success.log")