Skip to content
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

Commits are not properly attributed to the app bot #1

Open
armanbilge opened this issue Jun 2, 2022 · 2 comments
Open

Commits are not properly attributed to the app bot #1

armanbilge opened this issue Jun 2, 2022 · 2 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@armanbilge
Copy link
Owner

armanbilge commented Jun 2, 2022

Sadly it turns out that the 123456 in:

author-email: 123456+your-app-name[bot]@users.noreply.github.com

is not the App ID. It is in fact Yet Another ID that is assigned by GitHub to your app bot in an obfuscated manner.

The way we currently attribute commits made by GitHub Apps isn’t currently publicly documented in our Setting your commit email address article and is subject to change at anytime without notice.

Further reading:

@armanbilge armanbilge added bug Something isn't working help wanted Extra attention is needed labels Jun 2, 2022
armanbilge added a commit that referenced this issue Jun 2, 2022
@armanbilge
Copy link
Owner Author

armanbilge commented Jun 2, 2022

Ok, here's how you can get the ID, but only after the bot has opened some PRs using the wrong ID.

If you view the PR info via the GH REST API, then you can find your app bot's user ID under the user entry in the JSON. For example in:
https://api.github.com/repos/armanbilge/schrodinger/pulls/179
the nimble-agar-steward user ID is 106723251.

@armanbilge
Copy link
Owner Author

Yup, that fixes it. Compare armanbilge/schrodinger@fe802d8 (before) to armanbilge/gcp4s@f9307bf (after).

armanbilge added a commit to http4s/http4s that referenced this issue Jun 3, 2022
ckipp01 added a commit to ckipp01/scala-center-steward that referenced this issue Nov 29, 2022
NOTE: This isn't the same as the app id. You can see the context
for this in armanbilge/diy-steward#1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant