chore: Prune dependencies #544
Merged
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
Used in tests
Only used in an old test file, and we have an alternative we use everywhere else (chai-http)
Move packages required for runtime out of devDependencies
with the help of the depcheck npm package. Replaces some CI tasks dependencies with npx (npm package runner) Also pass --fix flag to eslint precommit hook.
Use git directly instead to write hash to file during docker build. The hash is already passed in dockerfile at build-time.
Less likely to change over time (compared to 'npm install'), which means better caching and faster build time. Requires to keep git around as it is used during npm install.
all about the deps, PR 544
MonkeyDo
force-pushed
the
prune-dependencies
branch
from
df2374e
to
ef0fa7f
Compare
Replaced with github action
MonkeyDo
force-pushed
the
prune-dependencies
branch
from
ac4c447
to
5b05aca
Compare
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.
Trying to improve build times by:
npx …which allows running a package without having it installed as a dependency in the project