search

tencent-connect / bot-node-sdk

QQ频道机器人 NODESDK
MIT License
103 stars 33 forks source link

修复内存泄露以及无限重连等问题 #43

Closed LittleBadBad closed 2 years ago

LittleBadBad commented 2 years ago
  1. 修复重连时重复监听导致的内存泄露问题
  2. 修复非网络故障导致掉线造成无限重连的问题
  3. 打包路径新增根目录参数
ostli commented 2 years ago

@LittleBadBad 目前存在冲突。请从最新的main分支，切出个人fix/xxx分支后，再提交新功能

另外 3 打包路径新增根目录参数，这个原因是啥？

LittleBadBad commented 2 years ago

@LittleBadBad 目前存在冲突。请从最新的main分支，切出个人fix/xxx分支后，再提交新功能

另外 3 打包路径新增根目录参数，这个原因是啥？

这个方便我在本地测试，需要打包到另外的路径

LittleBadBad commented 2 years ago

@LittleBadBad 目前存在冲突。请从最新的main分支，切出个人fix/xxx分支后，再提交新功能 另外 3 打包路径新增根目录参数，这个原因是啥？

这个方便我在本地测试，需要打包到另外的路径

是我要先提交到个人的fix/xxx分支，再提pr吗？

ostli commented 2 years ago

@LittleBadBad 目前存在冲突。请从最新的main分支，切出个人fix/xxx分支后，再提交新功能 另外 3 打包路径新增根目录参数，这个原因是啥？

这个方便我在本地测试，需要打包到另外的路径

这个本地测试的，不建议提交上来哈

ostli commented 2 years ago

@LittleBadBad 目前存在冲突。请从最新的main分支，切出个人fix/xxx分支后，再提交新功能 另外 3 打包路径新增根目录参数，这个原因是啥？

这个方便我在本地测试，需要打包到另外的路径

是我要先提交到个人的fix/xxx分支，再提pr吗？

1、从最新的main上切出fix/xxxx 2、在fix/xxx上修改、commit 3、pr

你是fork的，需要从我们的仓库更新到你的fork的仓库，保证你的main分支和我们的main是一致的.

fork的同步可参考：

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

LittleBadBad commented 2 years ago

@LittleBadBad 目前存在冲突。请从最新的main分支，切出个人fix/xxx分支后，再提交新功能 另外 3 打包路径新增根目录参数，这个原因是啥？

这个方便我在本地测试，需要打包到另外的路径

是我要先提交到个人的fix/xxx分支，再提pr吗？

1、从最新的main上切出fix/xxxx 2、在fix/xxx上修改、commit 3、pr

你是fork的，需要从我们的仓库更新到你的fock的仓库，保证你的main分支和我们的main是一致的.

fork的同步可参考：

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

ok