You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When using @capacitor/assets in an application using vite and vitest and not jest causes the editor to no longer flag the use of jest specific structures as an error, which degrades the DX when writing unit tests.
If I remove @capacitor/assets the errors are once again displayed in the editor.
Digging into this, I found that @capacitor/assets indirectly installs @types/jest:
Trapeze probably ought to have @types/jest in the devDependencies since it is a development time thing.
By the same token, though, I wasn't sure if Trapeze should be an installed dependency either as it also seems like a development time thing. I am guessing that both projects need their dependencies reviewed as to what belongs in the dependencies and what belongs in the devDependencies.
The text was updated successfully, but these errors were encountered:
Created PR #187 .
It was not clear to me which of the current deps really ought to be devDeps as I am not that familiar with the code base. However, it certainly does make sense that the @types packages would be devDeps.
When using
@capacitor/assets
in an application usingvite
andvitest
and notjest
causes the editor to no longer flag the use ofjest
specific structures as an error, which degrades the DX when writing unit tests.If I remove
@capacitor/assets
the errors are once again displayed in the editor.Digging into this, I found that
@capacitor/assets
indirectly installs@types/jest
:Trapeze probably ought to have
@types/jest
in thedevDependencies
since it is a development time thing.By the same token, though, I wasn't sure if Trapeze should be an installed dependency either as it also seems like a development time thing. I am guessing that both projects need their dependencies reviewed as to what belongs in the
dependencies
and what belongs in thedevDependencies
.The text was updated successfully, but these errors were encountered: