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

Fixed typo in sample configs #84

Merged
merged 1 commit into from
Feb 7, 2017

Conversation

simonellistonball
Copy link
Contributor

No description provided.

@janusgraph-bot
Copy link

Author of one or more commits is not listed as a CLA signer, either individual or as a member of an organization.

@janusgraph-bot janusgraph-bot added the cla: no This PR is not compliant with the CLA label Feb 4, 2017
Copy link

@amcp amcp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please submit a CLA

@mbrukman
Copy link
Member

mbrukman commented Feb 5, 2017

@simonellistonball — thanks for your contribution! Sorry for the inconvenience, but lawyers require us to have a CLA signed by every contributor (or their company) before accepting any contributions. For details, please see CONTRIBUTING.md for details.

If you plan to contribute to JanusGraph in the future (which we certainly hope you will), you'll have to sign a CLA regardless to cover all changes. If not, please feel free to close this PR, and we'll have to recreate this change ourselves.

If you or your legal counsel have any questions about the CLA, please send email to the email address in JanusGraph/legal/README.md.

@simonellistonball
Copy link
Contributor Author

@mbrukman would I not be covered under the Hortonworks corporate CLA?

@mbrukman
Copy link
Member

mbrukman commented Feb 5, 2017

@simonellistonball – the JanusGraph CLA verification process is specific to individuals (even in the presence of CCLAs) and precise in that it matches (name, GitHub username, email) for every commit.

If you're a Hortonworks employee (sorry, this is not at all obvious from either your GitHub profile or the email address used in your commits), please reach out to the existing Hortonworks employees listed in CLA_SIGNERS.yaml and have them update the Hortonworks section to include the required information for your account, at which point you'll be a recognized contributor. That file explicitly lists all users who are eligible to contribute to the JanusGraph repos — it covers all repos in the JanusGraph org.

The automated tool has no way of knowing that a contributor is employed by an organization which has already signed the CCLA unless the list of individuals on the CCLA is expanded to include more folks explicitly.

I'll start an email thread to discuss this.

@mbrukman mbrukman self-assigned this Feb 5, 2017
@ptgoetz
Copy link
Member

ptgoetz commented Feb 6, 2017

@simonellistonball has been added to the Hortonworks CCLA list.

@janusgraph-bot janusgraph-bot added cla: yes This PR is compliant with the CLA and removed cla: no This PR is not compliant with the CLA labels Feb 7, 2017
@mbrukman
Copy link
Member

mbrukman commented Feb 7, 2017

Thanks for the fix, @simonellistonball!

@mbrukman mbrukman merged commit 9f03be2 into JanusGraph:master Feb 7, 2017
micpod pushed a commit to micpod/janusgraph that referenced this pull request Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This PR is compliant with the CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants