Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Awilix V6 #272

Merged
merged 7 commits into from
Oct 31, 2021
Merged

Awilix V6 #272

merged 7 commits into from
Oct 31, 2021

Conversation

jeffijoe
Copy link
Owner

The next major version of Awilix is here, sporting a significant performance increase in some cases for container.resolve as well as a long-requested change to how dependencies for derived classes are resolved.

Closes #270, #198

/cc @code-ape @harindaka @linusnorton @bosconian-dynamics @udnes99

NOTE: Due to an issue with tslib, I had to disable CI builds for Node v17. See microsoft/tslib#134

This is to resolve any confusion as to what we
are checking existence for.

BREAKING CHANGE: This is changing the public API.
* Uses a recursive lookup to get the registration
 being resolved rather than doing a full copy of
 the registrations. This improves resolve
 performance by up to 1,000x in some cases
* Add `getRegistration` to container

Closes #270
BREAKING CHANGE: This breaks consumers that rely
 on the existing behavior where if a constructor was
 defined but with no parameters, the base class was
 parsed. Closes #198.
@coveralls
Copy link

coveralls commented Oct 31, 2021

Coverage Status

Coverage remained the same at 100.0% when pulling 13df4d5 on v6 into 0c4ee7d on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling f5625a8 on v6 into 0c4ee7d on master.

@jeffijoe jeffijoe merged commit 23ceef9 into master Oct 31, 2021
@jeffijoe jeffijoe deleted the v6 branch October 31, 2021 00:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Proposed fix for exponential performance slowdown due to dependencies count
2 participants