search

lxzan / gws

simple, fast, reliable websocket server & client, supports running over tcp/kcp/unix domain socket. keywords: ws, proxy, chat, go, golang...
https://pkg.go.dev/github.com/lxzan/gws
Apache License 2.0
1.36k stars 87 forks source link

Dev #14

Closed lesismal closed 1 year ago

lesismal commented 1 year ago

github action:. 增加 vet, lint, autobahn golang import: 标准库与三方库换行分隔 修复utils.StringToBytes的go vet告警 makefile all临时去掉 lint/test/bench/cover：我本地test会fail，lint告警比较多，我没全改了

只是尝试添加、触发action，不合并也可以，你多check下

lxzan commented 1 year ago

感谢

lxzan commented 1 year ago

goimports 不支持泛型吧, 本地运行 goimports -w . 一大堆报错, 看它仓库两年没更新了

lesismal commented 1 year ago

golang import: 标准库与三方库换行分隔

这个只是说每个文件import package的时候，标准库与非标准库分隔开，跟goimports没关系吧。

奇怪了，sh文件的执行权限没有生效，刚又改了Makefile里修改权限。 但是竟然也没有自动更新到这个pr里。我记得以前pr的时候，如果继续修改了src分支，会自动触发pr到dst分支这个里的更新。

lxzan commented 1 year ago

autobahn的报告要去哪里看?

lesismal commented 1 year ago

奇了怪了，autobahn的没跑起来，新开一个pr后旧的pr里也触发了更新

lesismal commented 1 year ago

本地make autobahn是ok的

lxzan commented 1 year ago

mac下面跑autobahn各种稀奇古怪的问题. 暂时不方便用台式机, 最近搬家被折腾死了

lesismal commented 1 year ago

搬家最烦了。 我再继续看看，慢慢搞

这里可以看日志： image

lesismal commented 1 year ago

github这个pr是真的有毒，我刚才把第二个pr关闭了，就又不触发自动push更新了

lxzan commented 1 year ago

github这个pr是真的有毒，我刚才把第二个pr关闭了，就又不触发自动push更新了

和目标分支有关系吗?

lxzan commented 1 year ago

实在不行就算了吧, 我还是喜欢在本地跑autobahn-testcase, 那些压缩项目太耗时了, 本地跑要快很多 https://lxzan.github.io/gws/reports/servers/

lesismal commented 1 year ago

恩，我先close了。

压缩耗时那个，压缩测试稳定没什么问题的话，可以把那几个压缩的测试去掉：

"exclude-cases": ["1[1-4].*"],

我之前本地跑带压缩的也很慢，好像至少要几十分钟

lesismal commented 1 year ago

和目标分支有关系吗?

不知道啊，怪不得linus喷github的merge垃圾呢，pr merge确实很恶心