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

Contributing graphql foundation #88

Merged

Conversation

mmatsa
Copy link
Contributor

@mmatsa mmatsa commented Aug 26, 2019

This PR has two changes to the libgraphqlparser CONTRIBUTING.md to clean up after
the move of the repo from Facebook to the GraphQL foundation.

One more reference to the contribution being to Facebook and not to the GraphQL
Foundation is still present in the CLA.

This PR already updates the license to point to the already-updated LICENSE and already
updates the security bug procedure to be reporting bugs to the project maintainers.

We are submitting this PR to start discussion about these two changes and also
the CLA.

This old reference to BSD must have been
accidentally forgotten when changing the
license previously.  Updating it in the same
way that it was updated for GraphiQL
repo, to point to the active current
LICENSE file.
The old text said to contact Facebook, back
from before this repo was transferred to the
GraphQL Foundation.  This updates that.

This does not yet update the CLA reference.
@leebyron leebyron merged commit 1244dce into graphql:master Aug 26, 2019
@leebyron
Copy link
Contributor

Thank you! I'll look for other incorrect references post repo ownership move and fix those as well.

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

Successfully merging this pull request may close these issues.

None yet

2 participants