search

wechaty / grpc

gRPC Service & Protocol Buffers for Wechaty Puppet
https://wechaty.github.io/grpc/
Apache License 2.0
26 stars 22 forks source link

Fix CI #201

Closed hcfw007 closed 1 year ago

hcfw007 commented 1 year ago

I checked the log of the last failed actions.

image

For C#, it failed to locate the google api proto files, so I renamed the files.

image

For Node, it failed for the same reason I explained in wechaty issue (https://github.com/wechaty/wechaty/issues/2551)

image

For PHP, it failed when checking protoc version. This is a bug of version check script. So I fixed it.

hcfw007 commented 1 year ago

@huan I believe I have fixed CI for PHP Node and C# (So all CI should pass now!), but somehow it won't trigger the pack process.

huan commented 1 year ago

Can you explain the modifications in the PR description? Because I want to understand what I will merge.

After that, the PR will be good to be merged.

hcfw007 commented 1 year ago

Can you explain the modifications in the PR description? Because I want to understand what I will merge.

After that, the PR will be good to be merged.

@huan Sure and done.

hcfw007 commented 1 year ago

Seems there are still some CI issue after this PR merged. The C# CI was good in the PR (https://github.com/wechaty/grpc/actions/runs/5676969240), but somehow failed now. The Node CI failed for Unknown compiler option 'verbatimModuleSyntax'. Maybe TS version was locked by onf of the dependencies. I'll look into them.

The Golang CI failed for acess right issue, maybe you can give it a look? @huan image

By the way, any idea of the reason Pack CI not triggered in PR?

huan commented 1 year ago

By the way, any idea of the reason Pack CI was not triggered in PR?

I think it's related to the bug in GitHub Actions.

Can you push a new commit to trigger it again to see if it works now?