Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

regro/cf-scripts

beckermr created a branch on regro/cf-scripts

beckermr-patch-2 - Flagship repo for cf-regro-autotick-bot

github-actions[bot] published a release on regro/cf-scripts
2024.10.4
## What's Changed * (Git-6) Use new Git Backend in Entire auto_tick flow by @ytausch in https://github.com/regro/cf-scripts/pull/3004 **Full Changelog**: https://github.com/regro/cf-scripts/compa...
github-actions[bot] created a tag on regro/cf-scripts

2024.10.4 - Flagship repo for cf-regro-autotick-bot

beckermr pushed 11 commits to main regro/cf-scripts
  • use git backend for entire auto_tick process added: open PR, comment on PR c120e1c
  • fix tests 80cf88f
  • use git's http.extraheader authentication instead of hiding tokens a0421fe
  • clear the token after pushing 1415cee
  • remove print_subprocess_output_strip_token 29b75a0
  • remove done TODO 1a6a13a
  • renamed mock b030c93
  • rename git_href to git_http_ref a 059014c
  • clear token in try...finally construct 2617679
  • introduce generic print_dry_run_message for printing dry run messages c37ea4f
  • Merge pull request #3004 from ytausch/continue-git-backend-part-6 (Git-6) Use new Git Backend in Entire auto_tick flow 27561c5

View on GitHub

beckermr closed a pull request on regro/cf-scripts
(Git-6) Use new Git Backend in Entire auto_tick flow
<!-- Thanks for contributing to cf-scripts! We are currently transitioning to a Pydantic-based model documenting the format of the conda-forge dependency graph data that this bot internally us...
beckermr created a review on a pull request on regro/cf-scripts

View on GitHub

beckermr created a review comment on a pull request on regro/cf-scripts
ah got it. SGTM and thanks for checking!

View on GitHub

beckermr created a review on a pull request on regro/cf-scripts

View on GitHub

ytausch created a comment on a pull request on regro/cf-scripts
Thank you! I just verified that no removed function is used in the regro or conda-forge organization via a quick GitHub search.

View on GitHub

ytausch created a review comment on a pull request on regro/cf-scripts
In the old code, the try-except block looks like this: https://github.com/regro/cf-scripts/blob/621910b1438b5ed2f3510fc72c7f692ac4e2ed5a/conda_forge_tick/auto_tick.py#L581-L601 In `push_repo`...

View on GitHub

ytausch created a review on a pull request on regro/cf-scripts

View on GitHub

ytausch created a review comment on a pull request on regro/cf-scripts
done

View on GitHub

ytausch created a review on a pull request on regro/cf-scripts

View on GitHub

ytausch created a review comment on a pull request on regro/cf-scripts
done in 0448fcd8ace879dc571364938603ecca7f2b168d, I called it `print_dry_run_message`

View on GitHub

ytausch created a review on a pull request on regro/cf-scripts

View on GitHub

ytausch created a review comment on a pull request on regro/cf-scripts
Done in 1a99cebbb5c491c95333344821ffd8a66b490a42.

View on GitHub

ytausch created a review on a pull request on regro/cf-scripts

View on GitHub

ytausch created a review comment on a pull request on regro/cf-scripts
agreed. 0ffe783fd9c3038043d9e4986cb83f01edcdfceb

View on GitHub

ytausch created a review on a pull request on regro/cf-scripts

View on GitHub

ytausch created a review comment on a pull request on regro/cf-scripts
b030c93bcd47e7902f66fd65dbca067a0dedf8a8

View on GitHub

ytausch created a review on a pull request on regro/cf-scripts

View on GitHub

ytausch created a review comment on a pull request on regro/cf-scripts
jup, removed in 1a6a13ac09be998f38f6cf675d037537dfa385d5

View on GitHub

ytausch created a review on a pull request on regro/cf-scripts

View on GitHub

ytausch created a review comment on a pull request on regro/cf-scripts
Yes, this was a note for myself :D code related to multi-token hiding removed in 29b75a07589ef0326a5ff250e70ae6d7f6599b8b

View on GitHub

ytausch created a review on a pull request on regro/cf-scripts

View on GitHub

beckermr created a review comment on a pull request on regro/cf-scripts
This needs to use a try...finally block for safety.

View on GitHub

beckermr created a review on a pull request on regro/cf-scripts

View on GitHub

beckermr created a review comment on a pull request on regro/cf-scripts
Maybe `get_http_ref` here or something? `get_href` confused me when I first read it.

View on GitHub

beckermr created a review on a pull request on regro/cf-scripts

View on GitHub

beckermr created a review comment on a pull request on regro/cf-scripts
Why are we no longer catching errors on the repo push?

View on GitHub

Load more