Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed

Conversation

LittleBadBad
Copy link

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

@ostli
Copy link
Contributor

ostli commented Jan 22, 2022

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

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

@LittleBadBad
Copy link
Author

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

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

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

@LittleBadBad
Copy link
Author

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

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

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

@ostli
Copy link
Contributor

ostli commented Jan 22, 2022

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

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

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

@ostli
Copy link
Contributor

ostli commented Jan 22, 2022

@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
Copy link
Author

@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

@LittleBadBad LittleBadBad deleted the master branch January 23, 2022 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants