Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: 修复在部分环境下,启动项目会缺失依赖的错误:Module not found: Error: Can't resolve 'spawn-sync' in '......' #376

Merged
merged 4 commits into from
Nov 7, 2024

Conversation

2361485765
Copy link
Contributor

@2361485765 2361485765 commented Oct 17, 2024

Ⅰ. 描述此PR的作用:

处理本地安装依赖遇到的问题以及添加缺乏的依赖
报错:Module not found: Error: Can't resolve 'spawn-sync' in '/Users/runner/work/dev-sidecar/dev-sidecar/packages/core/node_modules/shell-env/node_modules/cross-spawn'

Ⅱ. 此PR修复了哪个issue吗?

安装依赖遇到的报错如下截图所示,图为原来的依赖项中缺乏的一项
image

Ⅲ. 界面变化截屏

@2361485765 2361485765 closed this Oct 20, 2024
@wangliang181230 wangliang181230 changed the title build:修改搭建项目的依赖项 build: 修改搭建项目的依赖项 Nov 7, 2024
@wangliang181230 wangliang181230 changed the title build: 修改搭建项目的依赖项 build: 修复在部分环境下,启动项目会报错:Module not found: Error: Can't resolve 'spawn-sync' in '......' Nov 7, 2024
@wangliang181230
Copy link
Contributor

依赖调整,还是要的,我在Github Actions上,也碰到这个报错了:Module not found: Error: Can't resolve 'spawn-sync' in '/Users/runner/work/dev-sidecar/dev-sidecar/packages/core/node_modules/shell-env/node_modules/cross-spawn'

@wangliang181230 wangliang181230 changed the title build: 修复在部分环境下,启动项目会报错:Module not found: Error: Can't resolve 'spawn-sync' in '......' build: 修复在部分环境下,启动项目会缺失依赖的错误:Module not found: Error: Can't resolve 'spawn-sync' in '......' Nov 7, 2024
@wangliang181230
Copy link
Contributor

我把其他两个文件暂时撤回了:

  1. yarn 不同人的网络环境不同,提示在那npm失败用yarn就行了。
  2. background.js应该是存在的,不能改成 index.js

@wangliang181230 wangliang181230 merged commit 7f06be0 into docmirror:master Nov 7, 2024
wangliang181230 pushed a commit to wangliang181230/dev-sidecar that referenced this pull request Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants