code-review-bot
code-review-bot copied to clipboard
google
Code review bot and CLA verifier
For this issue: https://github.com/google/code-review-bot/issues/14 When PR is not compliant, instead of just leaving a comment, the bot would leave a "REQUEST_CHANGES" review with a message of the non-compliance.
In cases where there's a matching GitHub id in the CLA record, but a misspelling of contributor's name or email, we should provide a more specific error message to clarify...
E.g., when the user's GitHub id is empty (typically due to email address not being in the user's profile), CRBot should specify that, instead of failing with a generic error...
Right now, we just return a single value of whether the PR is compliant (yes/no) and whether or not its CLA is externally-managed. It would be better to return structured...
Since CRBot can add labels like `cla: yes`, `cla: no` and `cla: external` to a PR (if those labels exist), it would be nice to ensure both the existence of...
Right now, Code Review Bot simply adds a comment but does not block submission of the PR, so folks have to be careful to not accidentally merge a PR prior...
When committing code, users need to run: ``` git commit -s ``` which will add a line similar to the following at the end of the commit message: ``` Signed-off-by:...