InversifyJS icon indicating copy to clipboard operation
InversifyJS copied to clipboard

chore: upgrade dependencies

Open lifeiscontent opened this issue 3 years ago • 6 comments

  • 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.

lifeiscontent avatar Apr 14 '22 05:04 lifeiscontent

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

lifeiscontent avatar Apr 14 '22 05:04 lifeiscontent

CC: @PodaruDragos

lifeiscontent avatar Apr 16 '22 02:04 lifeiscontent

Codecov Report

Merging #1450 (27c179f) into master (c69c369) will decrease coverage by 0.00%. The diff coverage is n/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 data Powered by Codecov. Last update c69c369...27c179f. Read the comment docs.

codecov-commenter avatar Apr 16 '22 08:04 codecov-commenter

@lifeiscontent LGTM, thanks.

@jakehamtexas what do you think ?

Edit: re-running this again has failed. there are still some problems in the CI

PodaruDragos avatar Apr 16 '22 08:04 PodaruDragos

Maybe revert Sinon?

dcavanagh avatar Apr 17 '22 18:04 dcavanagh

@dcavanagh didn't seem to do it, any other ideas?

lifeiscontent avatar May 02 '22 21:05 lifeiscontent