-
Notifications
You must be signed in to change notification settings - Fork 65
chore(README): add easyCLA
#70
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Swastik Baranwal <[email protected]>
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.
lease add a commit message describing the reason for the change
I think the OP comment is enough. This PR just list another CLA to use. |
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.
Thanks Swastik! LGTM. I took a look at the repo and documentation to ensure it is still maintained and looks they have recent contributions to the main branch.
For future reference, and to highlight what @jerpelea shared in the comment above I think a brief description (not just of the tool, but why you believe it belongs in this list and section) would be helpful for the reviewers of this repo to better understand the context! 🙂
Signed-off-by: Ana Jimenez Santamaria <[email protected]>
I believe this can be merged now |
Adds EasyCLA a CLA service by LFX foundation