remove unused dependencies, move dev dependencies #128
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
lenkan
force-pushed
the
remove-unused-dependencies
branch
from
56394e8 to
ddefaaf
Compare
lenkan
force-pushed
the
remove-unused-dependencies
branch
from
ddefaaf to
8cd2ad4
Compare
Codecov Report
@@ Coverage Diff @@
## development #128 +/- ##
============================================
Coverage 80.94% 80.94%
============================================
Files 42 42
Lines 4173 4173
Branches 1028 1028
============================================
Hits 3378 3378
Misses 763 763
Partials 32 32 📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today! |
|
great clean up. Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Clean up dependencies as a preparation for a future npm publish. (#127)
Moved testing and build dependencies from "dependencies" to "devDependencies" to avoid having development dependencies shipped with published package. Also allows us to run
npm audit --omit=devto run audit on only runtime dependencies if we want.Remove dependencies not used from root package.json
Remove typescript and eslint dependencies from the examples/scripts and examples/integration-scripts packages. They will just use the typescript and ts-node packages from the root package using nodes module resolution. I think this is better to avoid having multiple versions of typescript installed here.
Upgrade typescript to 5.1. Latest is 5.1. The scripts used tsconfig settings that were not intruduced until v5, but tsdx or puppeteer had dependencies requiring <= 5.1.Note reverted this change because tsdx was not compatible with typescript v5 dues to outdated testing libraries. See also Consider moving away from tsdx #130