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

Fixup incorrect import #2581

Merged
merged 1 commit into from May 16, 2023
Merged

Fixup incorrect import #2581

merged 1 commit into from May 16, 2023

Conversation

pcmanus
Copy link
Contributor

@pcmanus pcmanus commented May 16, 2023

The DataRewrite.ts file was mistakenly importing the wrong assert method: it was importing the one from console, when it was meant to import the one from @apollo/federation-internals (as used everywhere else). This commit fixes that issue.

The `DataRewrite.ts` file was mistakenly importing the wrong `assert`
method: it was importing the one from `console`, when it was meant to
import the one from `@apollo/federation-internals` (as used everywhere
else). This commit fixes that issue.
@pcmanus pcmanus requested a review from a team as a code owner May 16, 2023 12:56
@netlify
Copy link

netlify bot commented May 16, 2023

👷 Deploy request for apollo-federation-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit f9d612f

@changeset-bot
Copy link

changeset-bot bot commented May 16, 2023

🦋 Changeset detected

Latest commit: f9d612f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@apollo/gateway Patch
@apollo/federation-internals Patch
@apollo/composition Patch
@apollo/query-planner Patch
@apollo/query-graphs Patch
@apollo/subgraph Patch
apollo-federation-integration-testsuite Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

@pcmanus pcmanus merged commit 772dd34 into apollographql:main May 16, 2023
10 checks passed
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