PicGo / PicGo-Core

:zap:A tool for pictures uploading. Both CLI & API supports.
https://picgo.github.io/PicGo-Core-Doc/
MIT License
832 stars 84 forks source link

希望增加一个模式,只输出最后上传成功的 URL #46

Open jwenjian opened 4 years ago

jwenjian commented 4 years ago

目前的 -s 或者 --silent 模式太 silent 了,都拿不到上传成功的 URL,而非 silent 模式又有一些其他日志,如果从这里拿上传的 URL,还要解析日志信息,希望可以增加一个新模式,只输出上传成功的 URL,对于多个图片,则一行一个。

这样方便在其他 cli 程序不集成 picgo node模块的情况下,获取到用 picgo 上传的图片 URL。

Molunerfinn commented 4 years ago

想法很好,不过细节有很多需要考虑,如果上传失败了,要不要显示?如果显示的话显示什么?如果不显示,那么你怎么知道是正在缓慢上传还是上传失败了?

目前来说你可以配置输出的日志的等级,目前默认是all,你只需要配置成日志只输出success和error我理解就能解决你的需求了

Molunerfinn commented 4 years ago

哦目前看了一下,日志等级配置只会影响picgo.log并不会影响控制台的输出。这块会在下个版本优化一下

Molunerfinn commented 4 years ago

最后的效果就类似客户端目前已经有的: image

jwenjian commented 4 years ago

好的,我去配置一下日志级别试试先

jwenjian commented 4 years ago

另外错误的时候感觉可以简单的输出error字符串,可以和图片URL区分开

jwenjian commented 4 years ago

依赖了 picgo 之后,运行的时候遇到了这个错误,是不是我的使用方法不对?

TypeError: this.db.set(...).write(...).catch is not a function
Molunerfinn commented 4 years ago

依赖了 picgo 之后,运行的时候遇到了这个错误,是不是我的使用方法不对?

TypeError: this.db.set(...).write(...).catch is not a function

是怎么运行的呢,在web里?

jwenjian commented 4 years ago

我有一个 cli 项目,依赖了 picgo,本地开发运行没问题,打包传到 npmjs 再通过 npm 安装之后,命令行里运行 就会报这个错

jwenjian commented 4 years ago

我基本确认了是 picgo 的相关代码报的,只要我没有用到上传图片的功能,就没有这个错。

我的 cli 在 https://github.com/jwenjian/giki-cli 希望帮忙看一下 多谢了

jwenjian commented 4 years ago

我的上传代码:https://github.com/jwenjian/giki-cli/blob/master/src/commands/new.ts#L21

Molunerfinn commented 4 years ago

无法复现- - image

Molunerfinn commented 4 years ago

哦可以复现了,把picgo的配置文件删掉就行

jwenjian commented 4 years ago

~/.picgo/config.json 吗?

Molunerfinn commented 4 years ago

嗯对。已经知道问题了,下个小版本会修复

jwenjian commented 4 years ago

好的

Molunerfinn commented 4 years ago

~/.picgo/config.json 吗?

对,有这个配置文件存在就不会报错

jwenjian commented 4 years ago

我现在是在 github action 里加了这句

mkdir -p ~/.picgo && echo '{"picBed":{"uploader":"smms","current":"smms","smms":{"token":""}},"picgoPlugins":{}}' > ~/.picgo/config.json

不然 npm publish 过不去.

现在 github action 能跑通了

Molunerfinn commented 4 years ago

更新了版本,已经不会有本issue的报错了