InversifyJS icon indicating copy to clipboard operation
InversifyJS copied to clipboard

Documentation about reflect-metadata is now misleading

Open terrybondy opened this issue 6 years ago • 1 comments

The documentation at https://github.com/inversify/InversifyJS/blob/master/wiki/environment.md contains the following note:

:warning: The reflect-metadata polyfill should be imported only once in your entire application because the Reflect object is mean to be a global singleton. More details about this can be found here.

Related issues in the dependency: #80 - metadata seemingly erased when using multiple files with karma,mocha,webpack #78 - Error when working with another polyfill, like core-js/reflect #45 - When installed redundantly, Metadata could be erased

seem to have been fixed (https://github.com/rbuckton/reflect-metadata/releases/tag/v0.1.12) in a previously used version of the dependency.

Expected Behavior

No note required.

Current Behavior

Has note.

Possible Solution

Remove note.

Steps to Reproduce (for bugs)

Context

Your Environment

  • Version used:
  • Environment name and version (e.g. Chrome 39, node.js 5.4):
  • Operating System and version (desktop or mobile):
  • Link to your project:

Stack trace

terrybondy avatar Aug 26 '19 14:08 terrybondy

Can't agree more. The newest version of reflect-metadata, which has been released for more than 4 year, can be imported more than once.

zanminkian avatar Nov 29 '22 04:11 zanminkian