-
Notifications
You must be signed in to change notification settings - Fork 92
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
Fix link to CLA in README.md #481
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.
Thank you for noticing this!
@ronnygunawan could you rebase this onto To do this locally, you should be able to do something like the following (assuming your remote name for this fork is
Then force-push the branch to your fork and edit this PR to target Sorry for the trouble! I've updated the default branch so this shouldn't happen again. |
@derrickstolee I've rebased it onto |
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 for doing the rebase! I'll merge this once the builds are green.
The builds aren't going because you have a previous green build that is causing the build to skip. I'll merge with admin privileges. Thanks again for your contribution! |
The link was broken because of an extra dot.
The link was broken because of an extra dot.
The link was broken because of an extra dot.