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

Remove unused cross-env dependency #197

Closed
jaller94 opened this issue Sep 5, 2019 · 0 comments · Fixed by #553
Closed

Remove unused cross-env dependency #197

jaller94 opened this issue Sep 5, 2019 · 0 comments · Fixed by #553
Labels
scope: dependencies Pull requests that update a dependency file

Comments

@jaller94
Copy link

jaller94 commented Sep 5, 2019

Current Behavior

´cross-env` is listed a s a dependency, but never used.
https://github.com/palmerhq/tsdx/blob/e942edbf9b74db611db9d4b095c5a742fdd97c99/package.json#L54

Desired Behavior

cross-env does not get listed as a dependency.

Suggested Solution

Remove cross-env from the list of dependencies.

Who does this impact? Who is this for?

All developers

Describe alternatives you've considered

Using cross-env, but the project does not seem to rely on env variables.

Additional context

This project has a scary amount of dependencies. 😨

@agilgur5 agilgur5 added the scope: dependencies Pull requests that update a dependency file label Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants