-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: set GH_REPO env var in Go license script #206
Conversation
I tried running this script for the first time this morning and it failed. Looking into how this works in the Copywrite tool led me to https://github.com/cli/go-gh#go-library-for-the-github-cli which got me hoping that maybe all this is missing is this `GH_REPO` env var. Might be a long shot, though.
[
|
[
|
[
|
[
|
mostly a success, I guess: https://github.com/cdktf/cdktf-repository-manager/actions/runs/5474604055 The fix works, at least |
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
[
|
Going to merge since this has been successfully run in all repos: https://github.com/cdktf/cdktf-repository-manager/actions/runs/5475602525 |
While working on #206 today, I discovered lots of small housekeeping-type things that could be improved with our actions. The most consequential of these is: https://github.com/cdktf/cdktf-repository-manager/blob/59b0ad5e300f71491f8411d9e530202cd8af5b66/.github/workflows/upgrade-repositories.yml#L99-L100 This will allow us to retry calls that fail due to rate limits, which might help alleviate the issue where sometimes, the PRs don't get merged because the "automerge" label didn't get added. Worth a try, anyway!
I'm going to lock this pull request because it has been closed for at least 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems related to this change, please open a new issue so we can investigate further. |
I tried running this script for the first time this morning and it failed. Looking into how this works in the Copywrite tool led me to https://github.com/cli/go-gh#go-library-for-the-github-cli which got me hoping that maybe all this is missing is this
GH_REPO
env var. Might be a long shot, though.