Skip to content

feat(build): added a sideEffects property for tree shaking package (#… #1682

feat(build): added a sideEffects property for tree shaking package (#…

feat(build): added a sideEffects property for tree shaking package (#… #1682

Triggered via push December 17, 2024 17:15
Status Success
Total duration 1m 4s
Artifacts

ci.yml

on: push
Fit to window
Zoom out
Zoom in

Annotations

12 warnings
Tests
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Verify Files
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Verify Files: demo/components/Playground.tsx#L50
Unexpected console statement
Verify Files: demo/components/Playground.tsx#L117
Unexpected console statement
Verify Files: demo/components/Playground.tsx#L118
Unexpected console statement
Verify Files: demo/components/Playground.tsx#L162
'directiveSyntax' is already declared in the upper scope on line 150 column 9
Verify Files: demo/components/Playground.tsx#L285
Unexpected console statement
Verify Files: demo/components/Playground.tsx#L289
Unexpected console statement
Verify Files: demo/components/Playground.tsx#L292
Unexpected console statement
Verify Files: demo/components/Playground.tsx#L312
React Hook useEffect has a missing dependency: 'props'. Either include it or remove the dependency array. However, 'props' will change when *any* prop changes, so the preferred fix is to destructure the 'props' object outside of the useEffect call and refer to those specific props inside useEffect
Verify Files: demo/components/SplitModePreview.tsx#L58
React Hook useMemo has an unnecessary dependency: 'theme'. Either exclude it or remove the dependency array
Verify Files: demo/hooks/useYfmHtmlBlockStyles.ts#L25
Assignment to property of function parameter 'acc'