add necessary packages to devDependencies

[aschlumpf]

Issue: #6

What Changed

Currently, there are dependencies which should be devDependencies in the external dependencies. This issue causes projects which use Storybook, using this package as a dependency, to install these packages. The @types packages which are installed are added to these projects' installed types, forcing dependencies to be type checked at these particular versions. There is no workaround built into TypeScript for installing transient@types from packages, see microsoft/TypeScript#18588.

This issue blocks my project from upgrading to Storybook 6.5.9, as lodash is having its types checked when we have version 4.17.21 installed and do not enforce strong typing on our use of lodash. The other changes are just good practice.

How to test

Run a build and run the tests.

Change Type

• maintenance
• documentation
• patch
• minor
• major

[kylegach]

@aschlumpf — Thank you for your (first! 🥳) contribution! I've assigned a relevant maintainer to take a look as soon as they're able.

[JannesMeyer]

Love this PR!

The prettier dependency "prettier": ">=2.2.1 <=2.3.0", is quite annoying and breaking stuff.

[PeerRich]

👍 this'll unlock some of our prettier issues ❤️

[emrysal]

This also closes #8 & #10 💯

[aschlumpf]

@shilman Any updates here for merging this in?

[trulyronak]

@shilman +1000 The prettier dependencies cause a bunch of problems, and there doesn't seem to be a way to override this. Would love to update prettier to 2.8.1 + keep it up to date!!

[dgattey]

This update to the prettier version would help immensely! The main @storybook packages use version 2.8.1, which unlocks some fixes around generics, namely enabling prettier to properly parse typeof x<Type> syntax, where you're passing a generic to a typeof. As it's just a dev dependency, merging this PR would enable us to not use an older version of prettier and instead correctly use 2.8.1 like we'd like to!

[marcovanharten-cpi]

Any update on this?

[aschlumpf]

@shilman Can we please try and get this merged soon? Thank you!

[chrisburkejr]

This looks good to me

[aschlumpf]

@chrisburkejr Thank you, it says I need approval from someone with write access in order to merge.