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

Use email for Github Actions bot #7

Merged
merged 1 commit into from
Aug 11, 2023
Merged

Use email for Github Actions bot #7

merged 1 commit into from
Aug 11, 2023

Conversation

jstvz
Copy link
Contributor

@jstvz jstvz commented Aug 10, 2023

Our company's tooling retrieves a list of commits for each pull request
and checks whether the commit's author has signed a Contributor License
Agreement. It filters bot accounts by checking the type property for
the string "Bot". However, commits made using the GITHUB_ACTIONS token
are showing up as type "User". This is WAD, however. The workaround
is to use the bot's ID to prefix the email address.

See: actions/checkout#1184

Our company's tooling retrieves a list of commits for each pull request
and checks whether the commit's author has signed a Contributor License
Agreement. It filters bot accounts by checking the `type` property for
the string ["Bot"]. However, commits made using the GITHUB_ACTIONS token
are showing up as `type` "User". This is [WAD], however. The workaround
is to use the bot's ID to prefix the email address.

See: actions/checkout/#1184

["Bot"]: https://github.com/salesforce/dr-cla/blob/290a3a1d/app/utils/GitHub.scala#L515-L519
[WAD]: https://github.com/orgs/community/discussions/26560#discussioncomment-3252337
@jstvz jstvz requested a review from a team as a code owner August 10, 2023 22:39
@jstvz jstvz merged commit c1d58fc into main Aug 11, 2023
1 check passed
@jstvz jstvz deleted the fix-cla-bot-errors branch August 11, 2023 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants