Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move @types/hoist-non-react-statics to devDependencies" #2310

Merged
merged 1 commit into from
Dec 17, 2024

Conversation

Erik-Gro
Copy link
Contributor

This pull request moves the "@types/hoist-non-react-statics" package from the dependencies section to the devDependencies section in the package.json file. This change is appropriate because type declaration files are typically used during development and compilation but are not required at runtime.

The modification ensures that the type definitions for "hoist-non-react-statics" are available during development and TypeScript compilation, but won't be included in the production build of projects using this package. This change helps to reduce the size of the production bundle and follows best practices for managing dependencies in TypeScript projects.
To implement this change, I used the following npm command:

npm install @types/hoist-non-react-statics --save-dev

This command installs the package as a dev dependency and automatically updates the package.json file, moving it from the "dependencies" section to the "devDependencies" section.

Copy link

socket-security bot commented Dec 16, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@types/hoist-non-react-statics@3.3.6 None 0 6.45 kB types
npm/jest@29.7.0 Transitive: environment, eval, filesystem, shell, unsafe +67 5.39 MB simenb

🚮 Removed packages: npm/@types/hoist-non-react-statics@3.3.4

View full report↗︎

@Erik-Gro Erik-Gro closed this Dec 16, 2024
@Erik-Gro Erik-Gro reopened this Dec 16, 2024
@adrai adrai merged commit e55b62d into i18next:master Dec 17, 2024
5 checks passed
@adrai
Copy link
Member

adrai commented Dec 17, 2024

thank you, it's included in v15.4.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants