Skip to content

Conversation

@michaelwittwer
Copy link
Member

No description provided.

- reflect-metadata should only exist once in the whole code base of a project so the consumer of our library must import the reflect-metadata project, even if he might not use it in his code base. But if he uses the reflect-metadata in his code or uses another library like InversifyJS we must ensure the Reflect global variable only exists once, to prevent from possible hard to debug issues
somehow the tslint is not working when using with staged files matching the test glob
- update README to provide some more information about what is required to start to use dynamo-easy
- adding comment with breaking change which I forgot to add previously

BREAKING CHANGE
all consumers of dynamo-easy must install and import reflect-metadata in their project.
@michaelwittwer
Copy link
Member Author

closes #84

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 76.202% when pulling 44c34c1 on #72-remove-reflect-metadata-import into bb9ab50 on beta.

@simonmumenthaler simonmumenthaler merged commit 85f098d into beta Nov 30, 2018
@michaelwittwer
Copy link
Member Author

🎉 This PR is included in version 4.0.1-beta.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@michaelwittwer michaelwittwer deleted the #72-remove-reflect-metadata-import branch November 30, 2018 20:35
@michaelwittwer
Copy link
Member Author

🎉 This PR is included in version 4.1.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@michaelwittwer
Copy link
Member Author

🎉 This PR is included in version 5.0.0-beta.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@michaelwittwer
Copy link
Member Author

🎉 This PR is included in version 5.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants