Unnecessary package.json dependencies about typegoose

[AmazingTurtle]

The reflect-metadata one is a bug.
Typegoose imports reflect-metadata. InversifyJS also requires it. So I imported it aswell.
I did not expect typegoose to import it internally aswell!

This caused me cancer and brainfuck.
If you use both InversifyJS and Typegoose, one must remove the own import of reflect-metadata or else it'll lead to weired errors.
Please leave the import of reflect-metadata to the users. Please.

[szokodiakos]

Hello. Sounds good, less dependencies mean less problems. If you have the time a PR of this would be greatly appreciated. If tests pass and the package remains completely usable then it is fine with me.

[Ben305]

We cannot remove the import because of https://github.com/szokodiakos/typegoose/blob/master/src/prop.ts#L176

[AmazingTurtle]

@Ben305 I see you're casting to any here. No strict typing. Therefore reflect-metadata is not required for passing a build.
I say: remove the import and leave it to the user, because other packages are importing reflect-metadata aswell - thus causing an error that's not so nice to resolve.

[Ben305]

We cannot remove it, because:

TypeError: Reflect.getMetadata is not a function
    at /Users/Ben305/Projects/typegoose/lib/prop.js:9:7841

Without casting to any, the typescript compiler throws an error.

I'd like to see a PR if you get it working.

[MichalLytek]

@Ben305
reflect-metadata is a polyfill, typegoose shouldn't import it by itself

[MichalLytek]

Closing this as it's out of date.