Skip to content

Commit 4cd6aaf

Browse files
committed
address review comments
1 parent 56b9a19 commit 4cd6aaf

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

homu/main.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -850,9 +850,10 @@ def init_local_git_cmds(repo_cfg, git_cfg):
850850
if not os.path.exists(fpath):
851851
utils.logged_call(['git', 'init', fpath])
852852

853-
remotes = {'origin': genurl(repo_cfg), 'test-origin': genurl(repo_cfg)}
854-
if 'test-on-fork' in repo_cfg:
855-
remotes['test-origin'] = genurl(repo_cfg['test-on-fork'])
853+
remotes = {
854+
'origin': genurl(repo_cfg),
855+
'test-origin': genurl(repo_cfg.get('test-on-fork', repo_cfg)),
856+
}
856857

857858
for remote, url in remotes.items():
858859
try:

homu/server.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -661,9 +661,10 @@ def report_build_res(succ, url, builder, state, logger, repo_cfg):
661661
def set_ref():
662662
utils.github_set_ref(state.get_repo(), 'heads/' +
663663
state.base_ref, state.merge_sha)
664-
utils.github_set_ref(state.get_test_on_fork_repo(),
665-
'heads/' + state.base_ref,
666-
state.merge_sha, force=True)
664+
if state.test_on_fork is not None:
665+
utils.github_set_ref(state.get_test_on_fork_repo(),
666+
'heads/' + state.base_ref,
667+
state.merge_sha, force=True)
667668
try:
668669
try:
669670
set_ref()

0 commit comments

Comments
 (0)