Closed dependabot[bot] closed 4 months ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
plugin-sandbox | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Feb 23, 2024 2:13pm |
yarn tries to keep as many dependencies as possible at the root node_modules folder. but sometimes some packages (within the monorepo) have to have their own node_modules folder when the package uses a different version of a certain package that is already in the root node_modules.
As we use dependabot and manage all the versions centrally, we've found a case that templates/js
package no longer need its own node_modules
folder. And we have a script to copy a temp version of vite config into node_modules
. And it failed because the folder didn't exist! (what a rare case)
Anyway, I've updated the script to create the folder if it's missing.
still getting type error. I ran into this in the previous PR, but don't remember how I fixed it 🥲 i'll get back to this tomorrow.
Looks like these dependencies are updatable in another way, so this is no longer needed.
Looks like these dependencies are updatable in another way, so this is no longer needed.
Why didn't you close this PR? I was working on it 😭
I've added skipLibCheck
to the root tsconfig.base.json
as it was failing with this error:
➜ yarn workspace @storyblok/field-plugin check:types
../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4127:14 - error TS2300: Duplicate identifier 'ElementType'.
4127 type ElementType = string | React.JSXElementConstructor<any>;
~~~~~~~~~~~
../../node_modules/@types/react/index.d.ts:4127:14
4127 type ElementType = string | React.JSXElementConstructor<any>;
~~~~~~~~~~~
'ElementType' was also declared here.
../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4141:14 - error TS2300: Duplicate identifier 'LibraryManagedAttributes'.
4141 type LibraryManagedAttributes<C, P> = C extends
~~~~~~~~~~~~~~~~~~~~~~~~
../../node_modules/@types/react/index.d.ts:4141:14
4141 type LibraryManagedAttributes<C, P> = C extends
~~~~~~~~~~~~~~~~~~~~~~~~
'LibraryManagedAttributes' was also declared here.
../../node_modules/@types/react/index.d.ts:4127:14 - error TS2300: Duplicate identifier 'ElementType'.
4127 type ElementType = string | React.JSXElementConstructor<any>;
~~~~~~~~~~~
../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4127:14
4127 type ElementType = string | React.JSXElementConstructor<any>;
~~~~~~~~~~~
'ElementType' was also declared here.
../../node_modules/@types/react/index.d.ts:4141:14 - error TS2300: Duplicate identifier 'LibraryManagedAttributes'.
4141 type LibraryManagedAttributes<C, P> = C extends
~~~~~~~~~~~~~~~~~~~~~~~~
../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4141:14
4141 type LibraryManagedAttributes<C, P> = C extends
~~~~~~~~~~~~~~~~~~~~~~~~
'LibraryManagedAttributes' was also declared here.
Found 4 errors in 2 files.
Errors Files
2 ../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4127
2 ../../node_modules/@types/react/index.d.ts:4127
node_modules
is obviously excluded from the config, but it seems there are some situations when this exclude
of tsconfig is not respected. So far, skipLibCheck
has been the only solution to this issue.
I've added
skipLibCheck
to the roottsconfig.base.json
as it was failing with this error:➜ yarn workspace @storyblok/field-plugin check:types ../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4127:14 - error TS2300: Duplicate identifier 'ElementType'. 4127 type ElementType = string | React.JSXElementConstructor<any>; ~~~~~~~~~~~ ../../node_modules/@types/react/index.d.ts:4127:14 4127 type ElementType = string | React.JSXElementConstructor<any>; ~~~~~~~~~~~ 'ElementType' was also declared here. ../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4141:14 - error TS2300: Duplicate identifier 'LibraryManagedAttributes'. 4141 type LibraryManagedAttributes<C, P> = C extends ~~~~~~~~~~~~~~~~~~~~~~~~ ../../node_modules/@types/react/index.d.ts:4141:14 4141 type LibraryManagedAttributes<C, P> = C extends ~~~~~~~~~~~~~~~~~~~~~~~~ 'LibraryManagedAttributes' was also declared here. ../../node_modules/@types/react/index.d.ts:4127:14 - error TS2300: Duplicate identifier 'ElementType'. 4127 type ElementType = string | React.JSXElementConstructor<any>; ~~~~~~~~~~~ ../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4127:14 4127 type ElementType = string | React.JSXElementConstructor<any>; ~~~~~~~~~~~ 'ElementType' was also declared here. ../../node_modules/@types/react/index.d.ts:4141:14 - error TS2300: Duplicate identifier 'LibraryManagedAttributes'. 4141 type LibraryManagedAttributes<C, P> = C extends ~~~~~~~~~~~~~~~~~~~~~~~~ ../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4141:14 4141 type LibraryManagedAttributes<C, P> = C extends ~~~~~~~~~~~~~~~~~~~~~~~~ 'LibraryManagedAttributes' was also declared here. Found 4 errors in 2 files. Errors Files 2 ../../node_modules/@types/react-dom/node_modules/@types/react/index.d.ts:4127 2 ../../node_modules/@types/react/index.d.ts:4127
node_modules
is obviously excluded from the config, but it seems there are some situations when thisexclude
of tsconfig is not respected. So far,skipLibCheck
has been the only solution to this issue.
This is interesting, i will try this locally tomorrow aswell if it's ok with you :)
LGTM @eunjae-lee.
I've tested it locally and it seems to be working nicely 💯
In the end, how the issue with
checking the node_modules
folder was solved? I ask because I saw theskipLibCheck
config was added but then removed and the error is not happening anymore 🤣 .
I'm not too sure exactly, but I assume there were duplicated libraries (probably different versions), and tsc failed with it. However, after removing yarn.lock and reinstalling dependencies, they lined up and now we use the same version, and the error no longer happens.
Bumps the everything group with 8 updates:
18.19.15
18.19.17
5.0.11
5.1.3
1.2.2
1.3.0
6.22.0
6.22.1
1.70.0
1.71.0
2.5.5
2.5.8
18.2.55
18.2.56
1.2.2
1.3.0
Updates
@types/node
from 18.19.15 to 18.19.17Commits
Updates
vite
from 5.0.11 to 5.1.3Release notes
Sourced from vite's releases.
Changelog
Sourced from vite's changelog.
... (truncated)
Commits
bf39527
release: v5.1.35099028
fix: cachedTransformMiddleware for direct css requests (#15919)4172f02
refactor(runtime): seal ES module namespace object instead of feezing (#15914)63a39c2
refactor(runtime): minor tweaks (#15904)75ddc0e
release: v5.1.27b15607
fix(runtime): pass path instead of fileURL toisFilePathESM
(#15908)d9adf18
chore: update license file (#15885)added3e
fix(build): do not output build time when build fails (#15711)642d528
chore(deps): update dependency dotenv-expand to v11 (#15875)d16ce5d
chore(deps): update all non-major dependencies (#15874)Updates
vitest
from 1.2.2 to 1.3.0Release notes
Sourced from vitest's releases.
... (truncated)
Commits
bb2af3b
chore: release v1.3.0bc5b2d0
fix: Prevent merging ofpoolOptions
(#5221)1aecd65
fix(vitest): fixoptimizeDeps.disabled
warnings on Vite 5.1 (#5215)b3182e0
chore: pass down--update
flag to the project config2ee2317
ci: add publish workflow (#5168)86297d4
fix(vitest): delegate snapshot options to workspace from root config (#5199)68f5196
fix: requires fixed version across the monorepo (#5208)7d9b1fb
feat(vitest): "test" accepts options object as the second parameter (#5142)656e210
fix(cli): parse--browser=\<name>
correctly (#5179)828858f
feat: deprecate watchExclude (#5171)Maintainer changes
This version was pushed to npm by vitestbot, a new releaser for vitest since your current version.
Updates
react-router-dom
from 6.22.0 to 6.22.1Release notes
Sourced from react-router-dom's releases.
... (truncated)
Changelog
Sourced from react-router-dom's changelog.
Commits
5f0cf32
chore: Update version for release (#11283)25d16db
chore: Update version for release (pre) (#11280)241f2d4
Fix issues with pre-encoded param names not being properly decoded (#11199)Updates
sass
from 1.70.0 to 1.71.0Release notes
Sourced from sass's releases.
Changelog
Sourced from sass's changelog.
Commits
3e6721e
Fix new static warnings with Dart 3.3 (#2173)2cab33e
Update the language revision in Homebrew on release (#2171)84ededd
Use musl support in cli_pkg (#2172)00571ec
Add a--pkg-importer
flag (#2169)84f31f0
Update pubspec/changelog forpkg:
importers (#2168)9ee5408
[Package Importer] Dart Implementation (#2130)9423aa5
Use macos-14 runner instead of macos-latest-xlarge runner (#2167)bbf97b4
Remove the sass dependency from package.json (#2162)Updates
@types/core-js
from 2.5.5 to 2.5.8Commits
Updates
@types/react
from 18.2.55 to 18.2.56Commits
Updates
@vitest/ui
from 1.2.2 to 1.3.0Release notes
Sourced from
@vitest/ui
's releases.... (truncated)
Commits
bb2af3b
chore: release v1.3.0581030e
fix(ui): fix tests duration time (#5219)2ee2317
ci: add publish workflow (#5168)68f5196
fix: requires fixed version across the monorepo (#5208)c28b4c2
feat(ui): save splitpanes size to local storage (#5166)e4e93df
chore: replace run-p with pnpm run (#5174)7a31a1a
chore(deps): update dependency@antfu/eslint-config
to v2 (#5153)29f1f79
chore(deps): update dependency floating-vue to v5 (#5148)16b3ab3
chore(deps): update dependency@vitejs/plugin-vue
to v5 (#5147)dcf0d6a
chore(deps): update antfu stylelint config (#5127)Maintainer changes
This version was pushed to npm by vitestbot, a new releaser for
@vitest/ui
since your current version.Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show