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

pnpm install 提示错误 #636

Closed
gavenwan opened this issue Apr 3, 2024 · 25 comments
Closed

pnpm install 提示错误 #636

gavenwan opened this issue Apr 3, 2024 · 25 comments
Labels
good first issue Good for newcomers

Comments

@gavenwan
Copy link
Contributor

gavenwan commented Apr 3, 2024

在部署的时候第一步pip install 没有报错,第二步 nom install -g 这里会 提示报错,分解这条命令,实际是最后一句 pnpm install 报错,提示 this project is configured by yarn

@yihong0618
Copy link
Owner

yarn install

@yihong0618
Copy link
Owner

我改下 readme

@yihong0618 yihong0618 added the good first issue Good for newcomers label Apr 3, 2024
@yihong0618
Copy link
Owner

这个该怎么改好一些。
改成 yarn 的原因是 vercel @orionna319

@gavenwan
Copy link
Contributor Author

gavenwan commented Apr 3, 2024

@gavenwan
Copy link
Contributor Author

gavenwan commented Apr 3, 2024

手动忽略错误,继续yarn run build ,会提示vite不是内部程序;
如果用pnpm develop 会提示,this project is configured by yarn

@gavenwan
Copy link
Contributor Author

gavenwan commented Apr 3, 2024

pack里去掉tachyons-sass,然后yarn run build ,在文件夹里手动添加css后,成功build运行

@yihong0618
Copy link
Owner

我今晚看看

@yihong0618
Copy link
Owner

谢谢反馈

@orionna319
Copy link
Collaborator

有ci错误链接地址吗,我本地yarn install && yarn build,都是成功执行的。
image

@orionna319
Copy link
Collaborator

是不是同步下最新的代码就好了

@yihong0618
Copy link
Owner

是不是同步下最新的代码就好了

我更新了下代码,#632 这个是因为 vercel 的问题。不知道是不是这个导致的

@gavenwan
Copy link
Contributor Author

gavenwan commented Apr 3, 2024

可能是,我手动改了package解决了问题就没有截图了。有空时我再试试重新下载最新包build一下。目前还在解决keep数据没能把我iphone 健身上的跑步同步完整的问题。

@yihong0618
Copy link
Owner

可能是,我手动改了package解决了问题就没有截图了。有空时我再试试重新下载最新包build一下。目前还在解决keep数据没能把我iphone 健身上的跑步同步完整的问题。

在issue里搜索,keep默认只有跑步。

@tianheg
Copy link
Collaborator

tianheg commented Apr 4, 2024

image

其实主要问题在于 tachyons 这个样式库,在 package.json 中不是通过版本号安装的,所以在 Vercel 上找不到。把 tachyons 相关代码删掉就行。

"tachyons-sass": "git+https://github.com/tachyons-css/tachyons-sass.git",

可以考虑换个样式库,比如 tailwind,但这是个大工程,短期内不太容易做到

@yihong0618
Copy link
Owner

image

其实主要问题在于 tachyons 这个样式库,在 package.json 中不是通过版本号安装的,所以在 Vercel 上找不到。把 tachyons 相关代码删掉就行。

"tachyons-sass": "git+https://github.com/tachyons-css/tachyons-sass.git",

可以考虑换个样式库,比如 tailwind,但这是个大工程,短期内不太容易做到

我们先删代码吧,有兴趣来个 PR 么?

@tianheg
Copy link
Collaborator

tianheg commented Apr 4, 2024

我们先删代码吧,有兴趣来个 PR 么?

因为很多 CSS 样式是基于 tachyons 的,所以只删除 tachyons 会导致页面没有样式,就像下面这样 :)
image

需要重写样式代码才能完成这个工作

@yihong0618
Copy link
Owner

我们先删代码吧,有兴趣来个 PR 么?

因为很多 CSS 样式是基于 tachyons 的,所以只删除 tachyons 会导致页面没有样式,就像下面这样 :) image

需要重写样式代码才能完成这个工作

OK 我找找有 fork 没

@yihong0618
Copy link
Owner

https://github.com/tachyons-css/tachyons-sass.git

我们可以 fork 一个自己发 npm 对吧?

@tianheg
Copy link
Collaborator

tianheg commented Apr 4, 2024

可以,这样只需要换这个包就行了

@yihong0618
Copy link
Owner

好的。你有兴趣搞一个么?

  • fork tachyons-css to tachyons-css-fork
  • new package tachyons-css-fork-${version}
  • push to npm
  • change the running_page tachyons-css package

@tianheg
Copy link
Collaborator

tianheg commented Apr 4, 2024

好的。你有兴趣搞一个么?

好呀,我现在就可以做

Update:

#639

@tianheg
Copy link
Collaborator

tianheg commented Apr 4, 2024

yihong 哥,还有个问题不太明白。Pnpm 用着不是挺好的吗,怎么就改用 Yarn 了。这个包管理器改来改去的,其实也没必要。

@yihong0618
Copy link
Owner

yihong 哥,还有个问题不太明白。Pnpm 用着不是挺好的吗,怎么就改用 Yarn 了。这个包管理器改来改去的,其实也没必要。

因为 vercel 报错。。。它自动用的 yarn

@tianheg
Copy link
Collaborator

tianheg commented Apr 4, 2024

那现在是用 Yarn 还是 Pnpm 呢,tachyons-sass 的问题解决了,Vercel 部署就没问题了吧,我自己测试过了没有错误
image

@yihong0618
Copy link
Owner

那现在是用 Yarn 还是 Pnpm 呢,tachyons-sass 的问题解决了,Vercel 部署就没问题了吧,我自己测试过了没有错误 image

我研究研究能一起用不。。

@tianheg tianheg closed this as completed Apr 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants