PicGo / vs-picgo

A VSCode plugin of PicGo
https://marketplace.visualstudio.com/items?itemName=Spades.vs-picgo
MIT License
375 stars 52 forks source link

:sparkles: Feature(error): throw error when no url found in output #101

Closed upupming closed 3 years ago

upupming commented 3 years ago

fixes issue #99 #100

upupming commented 3 years ago

@Spades-S 我最近在看 vue 的 release 脚本,你那边不是不太好加 MarketPlace 的发包权限吗,那能不能通过 API Token 实现自动发包,就像我打一个 tag,push 到 GitHub 之后,自动更新 GitHub Release 和 MarketPlace 的版本呢?

可以趁着这个机会把这个发包问题解决了

Spades-S commented 3 years ago

@Spades-S 我最近在看 vue 的 release 脚本,你那边不是不太好加 MarketPlace 的发包权限吗,那能不能通过 API Token 实现自动发包,就像我打一个 tag,push 到 GitHub 之后,自动更新 GitHub Release 和 MarketPlace 的版本呢?

可以趁着这个机会把这个发包问题解决了

token 放 git 里不安全,虽然是个开源的工具,但是也不要这么干 👻 ,商店发包这个事情还是手工干吧。

加发包权限申请流程太恶心了,如有需要,可以给一个 token,设置长一点的有效期

upupming commented 3 years ago

@Spades-S 可以放在仓库设置的secrets里面,这样GitHub action build的时候可以放在环境变量,js通过process env可以拿到的

upupming commented 3 years ago

你可以把token加到secret里面,我来写脚本

upupming commented 3 years ago

发现有现成的 action 可以直接用 https://github.com/lannonbr/vsce-action

Spades-S commented 3 years ago

你可以把token加到secret里面,我来写脚本

Done 🤗

upupming commented 3 years ago

新版本已经发布 @Spades-S @JeffersonQin ,v2.1.6: https://github.com/PicGo/vs-picgo/runs/3388738932?check_suite_focus=true

JeffersonQin commented 3 years ago

Cool!回家我做个测试,如果ok了close一下之前的issue

Spades-S commented 3 years ago

新版本已经发布 @Spades-S @JeffersonQin ,v2.1.6: https://github.com/PicGo/vs-picgo/runs/3388738932?check_suite_focus=true

@upupming

把 Release Tag 和 VSCode Marketplace Publish 解耦吧,避免频繁在商店更新。

约定下:

如何哇?

upupming commented 3 years ago

感觉这样可能会造成GitHub release和marketplace不一致

其实打tag不会特别频繁的,我是为了测试所以刚刚发了很多版本😊

code review 是不是应该在 merge 到 dev 的时候就做过了

我看 vue 的 npm 发包和 GitHub 发包也是同步的

Spades-S commented 3 years ago

感觉这样可能会造成GitHub release和marketplace不一致

其实打tag不会特别频繁的,我是为了测试所以刚刚发了很多版本😊

code review 是不是应该在 merge 到 dev 的时候就做过了

我看 vue 的 npm 发包和 GitHub 发包也是同步的

@upupming 在 merge 到 dev 的时候就 CR 确实会更好。如果要保持 release tag 和 商店版本一致的话,可以把 release tag 和 发布都放到 master 分支上。

现在的 master 分支 感觉好多余 😂