Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

build: upgrade apollo and graphql #72

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yharaskrik
Copy link

I did the root and express packages, dont know anything about fastify, koa or hapi but can take a look later

@s-r-x
Copy link
Owner

s-r-x commented Aug 17, 2023

@yharaskrik hi, thank you. looks good, at first glance. i have to run the code before merging in order to find any breaking changes, but unfortunately i don't have much time atm. I'll try to get my hands on it as soon as possible.

@s-r-x
Copy link
Owner

s-r-x commented Aug 27, 2023

@yharaskrik , so i checked the code and unfortunately i don't think that i'm going to merge it in the near future, because there are too many breaking changes, quite unpleasant ones.
"If you use Apollo Server with TypeScript, you must use TypeScript v4.7.0 or newer." - this one i don't like the most.

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

Successfully merging this pull request may close these issues.

2 participants