FaunaDB should be a dev dependency, not a regular one

[kripod]

As I’ve noticed in the latest canary release (3.2.0-canary.3), faunadb has been added as a dependency, but it doesn’t seem to be used besides a test. I would like to suggest adding faunadb to devDependencies while removing it from dependencies.

[balazsorban44]

We have something called peerOptionalDependencies, but honestly have no idea how we handle these, as it doesn't seem to be an npm standard. fauna should be added there I guess.

[kripod]

Yeah, I see your point, it should be implemented as outlined here, then: https://github.com/yarnpkg/rfcs/blob/master/accepted/0000-optional-peer-dependencies.md#2-detailed-design

[balazsorban44]

I think we use this at the moment: https://github.com/christkv/require_optional

[kripod]

Unfortunately, that doesn’t seem to be a living standard. However, the peerDependenciesMeta field is respected by Yarn and will eventually be adopted by npm if my memories are correct.

[balazsorban44]

cc @iaincollins

[balazsorban44]

https://docs.npmjs.com/cli/v7/configuring-npm/package-json#peerdependenciesmeta Looks like it is in npm@7 now!

[balazsorban44]

Fauna has been removed for now, see the explanation at #1146

[balazsorban44]

https://github.blog/2021-02-02-npm-7-is-now-generally-available/