Dependency on next-server

adamsoffer / next-apollo

React higher-order component for integrating Apollo Client with Next.js

MIT License

481 stars 64 forks source link

Dependency on next-server #30

Closed AndrewRayCode closed 5 years ago

AndrewRayCode commented 5 years ago

I'm not fully up to speed on the guts of next, but I'm using a custom express server with my next app. Installing next-server just to get execOnce seems like a lot? Context: I (apparently ) don't have next-server installed locally so when I changed next to a peerdependency in my fork of this project, and installed my fork locally, it didn't work.

adamsoffer commented 5 years ago

Yeah, we should definitely remove that dependency. Thanks for pointing this out!

adamsoffer commented 5 years ago

This dep has been removed as of v3.1.2