-
Notifications
You must be signed in to change notification settings - Fork 166
Add CLA bot GitHub action #3995
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| permissions: | ||
| contents: read | ||
| pull-requests: write | ||
| id-token: write # Needed to federate tokens. | ||
| actions: write |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grant issues:write so CLA bot can comment
The contributor-assistant step comments on pull requests to request or acknowledge CLA signatures, but the workflow’s explicit permissions block grants only contents, pull-requests, id-token, and actions. When permissions are declared, all others default to none, so the GITHUB_TOKEN lacks issues: write, which GitHub requires for creating PR/issue comments. On pull_request_target or issue_comment runs, the bot will fail with a 403 when trying to post the CLA reminder or confirmation, preventing the CLA check from functioning.
Useful? React with 👍 / 👎.
Bundles Sizes Evolution
🚀 CPU Performance
🧠 Memory Performance
|
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage 🔗 Commit SHA: 9175aa5 | Docs | Datadog PR Page | Was this helpful? Give us feedback! |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
Motivation
We are modifying the way we manage CLA signatures at Datadog
Changes
This PR introduces a new GitHub action to request contributors to sign Datadog's CLA by adding a comment to their first PR in the repository.
Test instructions
Once merged, I can test it by creating a no-op PR and sign the CLA myself
Checklist