Skip to content

Commit

Permalink
fix+doc: do not update pinnings and log actions (#3253)
Browse files Browse the repository at this point in the history
  • Loading branch information
beckermr authored Dec 1, 2024
1 parent 58e23dc commit ef473d9
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
3 changes: 2 additions & 1 deletion autotick-bot/install_bot_code.sh
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@ git config --global user.name regro-cf-autotick-bot
git config --global user.email [email protected]
git config --global pull.rebase false

conda update conda-forge-pinning --yes
# we pin everything now so no need to update this
# conda update conda-forge-pinning --yes

cd cf-scripts

Expand Down
6 changes: 6 additions & 0 deletions conda_forge_tick/events/pr_events.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,22 +40,26 @@ def react_to_pr(uid: str, dry_run: bool = False) -> None:
with pr_json:
pr_data = close_out_labels(pr_json, dry_run=dry_run)
if not dry_run and pr_data is not None and pr_data != pr_json.data:
print("closed PR due to bot-rerun label", flush=True)
updated_pr = True
pr_json.update(pr_data)

with pr_json:
pr_data = refresh_pr(pr_json, dry_run=dry_run)
if not dry_run and pr_data is not None and pr_data != pr_json.data:
print("refreshed PR data", flush=True)
updated_pr = True
pr_json.update(pr_data)

with pr_json:
pr_data = close_out_dirty_prs(pr_json, dry_run=dry_run)
if not dry_run and pr_data is not None and pr_data != pr_json.data:
print("closed PR due to merge conflicts", flush=True)
updated_pr = True
pr_json.update(pr_data)

if not dry_run and updated_pr:
print("pushed PR update", flush=True)
gh = github_client()
repo = gh.get_repo("regro/cf-graph-countyfair")
fpath = get_sharded_path(f"pr_json/{uid}.json")
Expand All @@ -67,3 +71,5 @@ def react_to_pr(uid: str, dry_run: bool = False) -> None:
dumps(pr_json.data),
cnts.sha,
)
else:
print("no changes to push", flush=True)

0 comments on commit ef473d9

Please sign in to comment.