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

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

Open
jwenjian opened this issue Jul 5, 2020 · 18 comments
Open

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

jwenjian opened this issue Jul 5, 2020 · 18 comments
Labels
enhancement New feature or request

Comments

@jwenjian
Copy link

jwenjian commented Jul 5, 2020

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

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

@Molunerfinn
Copy link
Member

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

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

@Molunerfinn
Copy link
Member

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

@Molunerfinn
Copy link
Member

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

@Molunerfinn Molunerfinn added the enhancement New feature or request label Jul 5, 2020
@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

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

@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

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

@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

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

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

@Molunerfinn
Copy link
Member

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

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

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

@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

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

@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

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

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

@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

@Molunerfinn
Copy link
Member

无法复现- -
image

@Molunerfinn
Copy link
Member

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

@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

~/.picgo/config.json 吗?

@Molunerfinn
Copy link
Member

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

@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

好的

@Molunerfinn
Copy link
Member

~/.picgo/config.json 吗?

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

@jwenjian
Copy link
Author

jwenjian commented Jul 5, 2020

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

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

不然 npm publish 过不去.

现在 github action 能跑通了

@Molunerfinn
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants