Skip to content

feat: Support @apollo/client@^4#86

Merged
MasterOdin merged 11 commits into
masterfrom
apollo-4-2
Dec 26, 2025
Merged

feat: Support @apollo/client@^4#86
MasterOdin merged 11 commits into
masterfrom
apollo-4-2

Conversation

@MasterOdin

@MasterOdin MasterOdin commented Dec 26, 2025

Copy link
Copy Markdown
Collaborator

Closes #82

PR adds support for @apollo/client@4 to the library. This is similar to #85, though just utilizes the deprecated types only, which allows for a somewhat cleaner experience for folks on v3 without having to worry about skipLibCheck tsconfig setting.

However, we still move to using es2016 for compilation vs es5, however, the same point still holds where:

Luckily @apollo/client@3 happily works with either. While this is a breaking change, per https://node.green/, anyone using node 6+ is compatible with it, where node 6 was released 9 years ago and hit end of life 7 years ago. At this point, I think anyone still using such an old version of node is also probably not bothering to update their libraries either.

Add to this that the library only tests against 12.x, then this shouldn't affect anyone meaningfully that I think ti'd be fine to put this out as a point release.

Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
Signed-off-by: Matthew Peveler <matt.peveler@gmail.com>
Comment thread .github/workflows/ci.yml
node-version: ${{ matrix.node-version }}

- run: yarn --ignore-scripts
- run: yarn add --ignore-scripts --dev @apollo/client@4.0.0 graphql@16.0.0 rxjs@^7.3.0

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

7.3.0 is the peer dependency for @apollo/client, but it doesn't compile against cjs. Not really keen to try to figure out the lowest version, and not sure it really matters vs @apollo/client which is the real thing we build against.

@MasterOdin MasterOdin marked this pull request as ready for review December 26, 2025 00:50
@MasterOdin MasterOdin merged commit c92cd46 into master Dec 26, 2025
13 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.

Support @apollo/client version 4

1 participant