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

"body-parser" should not be dev-dependency #108

Open
bkilinc opened this issue Sep 29, 2020 · 2 comments
Open

"body-parser" should not be dev-dependency #108

bkilinc opened this issue Sep 29, 2020 · 2 comments

Comments

@bkilinc
Copy link

bkilinc commented Sep 29, 2020

"body-parser" is in "devDependencies" section. It should be in "dependencies" section.

@Uzlopak
Copy link

Uzlopak commented Oct 29, 2020

I disagree, as the body-parser is not needed at all.

@paularah
Copy link

I share the same opinion with @Uzlopak I think body-parser as an external dependency has been deprecated from express, so there's no need for it.

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

No branches or pull requests

3 participants