-
Notifications
You must be signed in to change notification settings - Fork 3
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
Gracefully handle lack of CLA for Gerrit user #22
Comments
May not be necessary with the correct group settings... checking... |
This could potentially be handled/linked with #25 |
Yes, this is expected if the account hasn't been added to one of the CLA groups.
Definitely should handle this gracefully and add a proper annotation to the job run instead of requiring folks to troll though job log. The annotation should basically be |
Hmmmm, this is perhaps expected with our Gerrit setup:
However, it doesn't look like we're handling the error gracefully.
We should probably put a fix in place.
This was captured from the workflow run here
The text was updated successfully, but these errors were encountered: