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.

Sign up for GitHub

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

Jump to bottom

apollo-server-core(usageReporting): remove reference to apollo-graphql #5981

Merged
merged 1 commit into from
Dec 31, 2021
Merged

apollo-server-core(usageReporting): remove reference to apollo-graphql #5981

merged 1 commit into from
Dec 31, 2021

Conversation

sushantdhiman
Copy link
Contributor

Fixes #5980

Remove reference to now removed dependency apollo-graphql from usageReporting plugin, use inlined defaultUsageReportingSignature instead.

@apollo-cla
Copy link

@sushantdhiman: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

@glasser
Copy link
Member

glasser commented Dec 31, 2021

Thanks!

@glasser glasser merged commit 7679bef into apollographql:main Dec 31, 2021
@sushantdhiman sushantdhiman deleted the patch-2 branch January 1, 2022 03:59
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[email protected] is trying to import now removed apollo-graphql dependency
3 participants
@sushantdhiman @apollo-cla @glasser