Skip to content

feat(ci): Use GitHub App token instead of default token#7680

Merged
philprime merged 1 commit intomainfrom
philprime/updater-app
Mar 12, 2026
Merged

feat(ci): Use GitHub App token instead of default token#7680
philprime merged 1 commit intomainfrom
philprime/updater-app

Conversation

@philprime
Copy link
Member

Switch the Auto Update Tools workflow to authenticate with the configured GitHub App instead of the default GITHUB_TOKEN.

Dependency update PRs (like #7675) will now be created and updated using the SENTRY_DEPENDENCY_UPDATER_GITHUB_APP_* credentials. This ensures PRs and commits are attributed to the app rather than the default workflow token.

The workflow generates and uses the app token for checkout and both create-pull-request steps (clang-format and swiftlint).

This applies the same fix from getsentry/sentry-fastlane-plugin#429 to this repository.

#skip-changelog

Switch the Auto Update Tools workflow to authenticate with the
configured GitHub App instead of the default GITHUB_TOKEN.

Dependency update PRs (like #7675) will now be created and updated
using the SENTRY_DEPENDENCY_UPDATER_GITHUB_APP_* credentials. This
ensures PRs and commits are attributed to the app rather than the
default workflow token.

The workflow generates and uses the app token for checkout and both
create-pull-request steps (clang-format and swiftlint).
@github-actions
Copy link
Contributor

github-actions bot commented Mar 12, 2026

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@philprime philprime self-assigned this Mar 12, 2026
@github-actions
Copy link
Contributor

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 29a280c

@philprime philprime added the ready-to-merge Use this label to trigger all PR workflows label Mar 12, 2026
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Copy link
Contributor

@itaybre itaybre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, AI comments don't make sense here

@philprime philprime merged commit c9e5b2d into main Mar 12, 2026
185 of 198 checks passed
@philprime philprime deleted the philprime/updater-app branch March 12, 2026 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Use this label to trigger all PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants