Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Commit

Permalink
deps: Organize dependencies (#2)
Browse files Browse the repository at this point in the history
* deps: Move development dependencies to devDependencies

* deps: Remove unused dependencies
  • Loading branch information
ardalanamini authored Mar 6, 2022
1 parent cc9d325 commit e9f1a07
Show file tree
Hide file tree
Showing 3 changed files with 246 additions and 2,686 deletions.
5 changes: 4 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,13 @@ This section includes the issues, changes & improvements I've made, with the tho
> as it makes it difficult to debug & avoid possible incompatibility & security issues in dependencies.
- Missing `"private": true` property:
> This property will avoid accidental publication of the repository.
- `main` property pointing to a non-existing file.
- `devDependencies` being wrongly included in `dependencies`:
> This will result to unnecessary packages being installed in the production server.
> This can increase production project size (e.g. Docker image).
- `main` property pointing to a non-existing file.
> Related dependencies: `@types/cors`, `@types/express`, `@types/express-handlebars`, `ts-node`, `ts-node-dev`, `typescript`
- Unused dependencies.
> Related dependencies: `chart.js`, `express-handlebars`, `lodash`, `luxon`, `@types/express-handlebars`, `eslint-plugin-react`, `react`
- Issues in the `src` directory:
- Unused imports. (e.g. unused `lodash` import in the `src/scripts/seed.ts`)

Expand Down
Loading

0 comments on commit e9f1a07

Please sign in to comment.