fjc0k / docker-YApi

接口管理平台 YApi 的 Docker 镜像。
https://hub.docker.com/r/jayfong/yapi/tags
MIT License
1.09k stars 261 forks source link

支持docker-compose直接挂载插件包 #81

Closed cabbage89 closed 1 year ago

cabbage89 commented 2 years ago

当前插件必须上架到yarn商店才能安装,yarn本身是支持 yarn add file:<localpath> 安装的, 增加判断用于兼容

volumes: 
      - /home/xxxxxx/yapi-plugin-hello-master:/yapi/vendors/node_modules/yapi-plugin-hello
fjc0k commented 2 years ago

这个目前是支持的吧,不需要这么改,本来这个就剔除了本地存在的不通过yarn安装:

        .filter(
          (packageName) =>
            !fs.existsSync(`/yapi/vendors/node_modules/${packageName}`),
        )

那既然这样,你通过 config 配置的并且通过 volume 映射进去的本地插件就不会被覆盖。

因为 node 查找包时并不会检查 package.json 有没有,因此只需映射进去就行,不需要安装。

fjc0k commented 2 years ago

我的锅,仔细想了下,这个PR是有必要的。