move eslint into devDependencies #18

Open
opened 2022-08-01 12:06:59 +00:00 by floatingghost · 2 comments

just a gripe

the dependencies field in package.json informs dependents of what they should download, and having stuff like eslint in there likely conflicts with what they use

dependencies should only list stuff needed at runtime, the rest can go in devDependencies

just a gripe the `dependencies` field in package.json informs dependents of what they should download, and having stuff like eslint in there likely conflicts with what they use dependencies should only list stuff needed at runtime, the rest can go in devDependencies
Owner

i did do that, but it just put it in regular dependencies
i found that weird as well but thought that maybe because it's an extension that uses peerDependencies it's a special case, i dunno

i did do that, but it just put it in regular dependencies i found that weird as well but thought that maybe because it's an extension that uses peerDependencies it's a special case, i dunno
Author

weird - well anyhow it's not critical, just a possible thingy

weird - well anyhow it's not critical, just a possible thingy
Sign in to join this conversation.
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: nbsp/marked-mfm#18
No description provided.