Update csharp.yml - Githubissues

wechaty / grpc

gRPC Service & Protocol Buffers for Wechaty Puppet

https://wechaty.github.io/grpc/

Apache License 2.0

26 stars 22 forks source link

Open Jesn opened 2 years ago

Jesn commented 2 years ago

csharp action workflow-> remove push to nuget job

huan commented 2 years ago

Could you please have a short explanation about this change?

Thank you very much!

Jesn commented 2 years ago

csharp.yml 之前会build csharp/Wechaty.Grpc 解决方案生成nuget包，并推送到Nuget服务器，本次修改移除了Nuget包推送到Nuget服务器的这一步操作，只验证解决方案是否报异常

huan commented 2 years ago

Thanks for the clarification!

According to our best practice of DevOps, I think it would be great if we could publish our package to the Nuget server after the CI passed.

However, this is a suggestion and you will make the final decision.

Should we merge this PR for now?

huan commented 1 year ago

ping @Jesn

CLAassistant commented 11 months ago

All committers have signed the CLA.