Skip to content

ci(update): use app token instead of obsolete PAT#1013

Merged
gr2m merged 1 commit into
mainfrom
gr2m-patch-1
Apr 23, 2025
Merged

ci(update): use app token instead of obsolete PAT#1013
gr2m merged 1 commit into
mainfrom
gr2m-patch-1

Conversation

@gr2m

@gr2m gr2m commented Apr 23, 2025

Copy link
Copy Markdown
Contributor

This should resolve the failing Update workflow builds: https://github.com/octokit/graphql-schema/actions/workflows/update.yml

image

@github-actions

Copy link
Copy Markdown
Contributor

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

@wolfy1339

wolfy1339 commented Apr 23, 2025

Copy link
Copy Markdown
Member

Might as well update these as well

GITHUB_TOKEN: ${{ secrets.OCTOKITBOT_PAT }}

GITHUB_TOKEN: ${{ secrets.OCTOKITBOT_PAT }}

@gr2m

gr2m commented Apr 23, 2025

Copy link
Copy Markdown
Contributor Author

Might as well update these as well

GITHUB_TOKEN: ${{ secrets.OCTOKITBOT_PAT }}

let's do follow up once we see this works

@gr2m gr2m merged commit 6bb8d3a into main Apr 23, 2025
@gr2m gr2m deleted the gr2m-patch-1 branch April 23, 2025 20:22
@github-project-automation github-project-automation Bot moved this from 🆕 Triage to ✅ Done in 🧰 Octokit Active Apr 23, 2025
@wolfy1339

Copy link
Copy Markdown
Member

I've replaced OCTOKITBOT_TOKEN in other repositories with this solution, and it works fine
As outlined in https://github.com/octokit/maintainers/issues/34

@gr2m

gr2m commented Apr 23, 2025

Copy link
Copy Markdown
Contributor Author

okay that worked. Sorry I just got most annoyed by this specific workflow because I got a notification for each build error

@gr2m

gr2m commented Apr 23, 2025

Copy link
Copy Markdown
Contributor Author

ping me if you need a review for more, I need to dive into my other work

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 15.26.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants