InversifyJS
InversifyJS copied to clipboard
chore: upgrade dependencies
- upgrade dev dependencies
- make main build command faster by parallelizing the work across formats
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Types of changes
- [x] Updated docs / Refactor code / Added a tests case (non-breaking change)
- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
- [x] My code follows the code style of this project.
- [ ] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
- [x] I have read the CONTRIBUTING document.
- [ ] I have added tests to cover my changes.
- [x] All new and existing tests passed.
- [x] I have updated the changelog.
side notes:
- I noticed there's still a .travis.yml in the repo, I can delete it if you guys want.
- should probably add Node 16 to GitHub Action as it's the LTS version
CC: @PodaruDragos
Codecov Report
Merging #1450 (27c179f) into master (c69c369) will decrease coverage by
0.00%. The diff coverage isn/a.
@@ Coverage Diff @@
## master #1450 +/- ##
==========================================
- Coverage 99.85% 99.85% -0.01%
==========================================
Files 53 53
Lines 1367 1361 -6
Branches 192 192
==========================================
- Hits 1365 1359 -6
Partials 2 2
| Impacted Files | Coverage Δ | |
|---|---|---|
| src/resolution/instantiation.ts | 98.30% <0.00%> (-0.03%) |
:arrow_down: |
| src/planning/planner.ts | 100.00% <0.00%> (ø) |
|
| src/planning/request.ts | 100.00% <0.00%> (ø) |
|
| src/resolution/resolver.ts | 100.00% <0.00%> (ø) |
|
| src/utils/binding_utils.ts | 100.00% <0.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update c69c369...27c179f. Read the comment docs.
@lifeiscontent LGTM, thanks.
@jakehamtexas what do you think ?
Edit: re-running this again has failed. there are still some problems in the CI
Maybe revert Sinon?
@dcavanagh didn't seem to do it, any other ideas?