csstype icon indicating copy to clipboard operation
csstype copied to clipboard

Test against third-party libraries

Open frenic opened this issue 7 years ago • 6 comments

Even small changes may have impact on third-party libraries that uses CSSType (like JSS, Styletron and Glitz) and tests against these would provide safety when publishing new releases. Not sure about the best approach to keep it quite general for both TypeScript/Flow and when API/definitions/tests changes on these libraries? It shouldn't fail unless a change in CSSType is the cause of the failure.

frenic avatar Mar 03 '18 13:03 frenic

Add React to that list now!

pelotom avatar Apr 03 '18 21:04 pelotom

Yeah, we don't want that to happen again 😅 I'll do it manually until I figure out a way to automate it.

frenic avatar Apr 03 '18 23:04 frenic

Hm, it seems like it doesn't work just by linking csstype and run yarn test locally with DefinitelyTyped. It seems like dtslint makes clean installations of NPM dependencies before linting. yarn lint react works though but I would like to cover other packages that depends on React.CSSProperties as well.

frenic avatar Apr 04 '18 18:04 frenic

Please consider adding Material-UI to the list.
I know it's indirect, via JSS, but still.

estaub avatar Apr 12 '18 19:04 estaub

@estaub I know you had troubles when the JSS types was changed. But those were/are directly due to @types/jss and not csstype as I understand. Does material-ui have any types that relies on the properties and values from csstype? Could you point me into the right direction so I can get an understanding of which changes in csstype could possibly affect material-ui?

frenic avatar Apr 12 '18 21:04 frenic

Well, there's the &$%&%$ type-widening issue, that needs some kind of usability bandaid, at a minimum. See https://github.com/DefinitelyTyped/DefinitelyTyped/issues/24952#issuecomment-380936904, where I try to offer some options.

estaub avatar Apr 12 '18 22:04 estaub