PicGo-Core icon indicating copy to clipboard operation
PicGo-Core copied to clipboard

picgo dist 不应包含 src 目录

Open upupming opened this issue 4 years ago • 2 comments

现在编译出来的 dist 包含了 src 目录,这样做的目的是什么呢?是不是因为需要 -v 查看版本号引用 package.json 所以保留了原始的 src 目录结构?

image

这样做会造成 import 的时候出现这样的语句:

https://github.com/PicGo/vs-picgo/blob/e8b30b72f5c68a0a31ea98302603968f85ab93bb/src/utils/index.ts#L4

总感觉这样有点奇怪,又有 dist 又有 src,语义上有点歧义。

或者说 vs-picgo 在引用的时候是不是应该直接 import src 文件夹下的内容(import { ImgInfo } from 'picgo/src/utils/interfaces';)?但是目前 picgo 的 src 目录是 npmignore 掉了的,npm install 之后没有 src 文件夹。

upupming avatar Jun 19 '20 14:06 upupming

tsc在编译的时候由于会用到 package.json 所以会把 src 目录也带上才能拥有正确的层级。(tsc只负责编译但是不负责打包)

这点之前是比较头疼的。之后换成rollup打包应该能解决这个问题。

Molunerfinn avatar Jun 23 '20 02:06 Molunerfinn

要改需要发个大版本标明breaking change。不过目前来看暂时先别动

Molunerfinn avatar Jun 23 '20 02:06 Molunerfinn

done

Molunerfinn avatar Nov 15 '22 14:11 Molunerfinn