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

Add entries for cppgraphqlgen to the C/C++ section #1612

Open
wants to merge 5 commits into
base: source
Choose a base branch
from

Conversation

wravery
Copy link

@wravery wravery commented Jan 23, 2024

Description

Both of these libraries/tools are part of the microsoft/cppgraphqlgen project, but I split them into separate entries to make it clear that it supports generating both client and server implementations in C++.

The markdown looks fine to me in a previewer, but I haven't been able to get the site to run the "Code" page locally (with or without my addition) to verify these changes look OK on the site. It throws an error trying to dereference an undefined data variable on my machine. If there are any problems with the formatting or the way this looks on a working build, please feel free to send it back for fixes.

Thanks!

Copy link

vercel bot commented Jan 23, 2024

@wravery is attempting to deploy a commit to the The GraphQL Foundation Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

linux-foundation-easycla bot commented Jan 23, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@benjie
Copy link
Member

benjie commented Jan 23, 2024

@wravery I believe Microsoft already have a CLA in place with GraphQL Foundation, so you need to reach out to whatever internal team manages that and have your username added to it. Once done, click the link in the EasyCLA comment above to have it re-evaluate.

@wravery
Copy link
Author

wravery commented Jan 23, 2024

@benjie Thanks! I am waiting to hear back from the team that I think would manage this CLA, I'll trigger the reevaluation once I do.

@wravery
Copy link
Author

wravery commented Feb 23, 2024

@benjie just bumping this for awareness since I've signed the CLA and reevaluated the check.

After consulting with the team that EasyCLA knew about and their legal contact, they recommended that I just sign it as a personal CLA. There is no central oversight for this particular CLA at Microsoft, nor did they want to take on that responsibility.

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