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

Feature Request: Endpoint to query CLA status for a given commit #3380

Closed
mrbobbytables opened this issue Nov 30, 2021 · 4 comments
Closed
Labels

Comments

@mrbobbytables
Copy link

Summary

An endpoint that returns the EasyCLA status for a given commit, preferably with additional information to troubleshoot (e.g. email, individual/corp+who if corp CLA etc)

Background

Multiple issues have been encountered during the Kubernetes EasyCLA migration where it is not clear what is wrong. For example a PR from a single author with multiple commits where some of the commits are throwing a bad EasyCLA status. There is no indication as to why it thinks it's wrong, which kind of CLA the person has signed or the employer associated with it.

Tracking issue here: kubernetes/org#3068

User Story

  • As a contributor I would like to understand why EasyCLA believes I have not signed a CLA

Acceptance Criteria

When an endpoint returns the CLA status for a given commit
When an endpoint returns the information for a given commit

References

Kubernetes Org Migration Feedback thread: kubernetes/org#3068

@dealako
Copy link
Member

dealako commented Feb 7, 2023

Example of a confusing commit status:

Screenshot 2023-02-07 at 7 50 01 AM

Observations

  1. Each commit should be a link represented in blue.
  2. Why is the first commit a short SHA value, while the others are long?

@dealako
Copy link
Member

dealako commented Feb 7, 2023

This is reported a long time ago. I know we did some updates on the status update logic. I wonder if this issue continues to exist.

@nickmango
Copy link
Collaborator

@mrbobbytables this is not reproducible currently. Kindly confirm
cc @mlehotskylf

@mlehotskylf
Copy link
Contributor

This i fixed, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants