-
-
Notifications
You must be signed in to change notification settings - Fork 65
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
feat: awaitable connect #55
Conversation
BREAKING CHANGE: ClientConnect replaced with ClientConnectAsync that can be awaited
Nice PR, but why did you modified ServerDisconnect too? |
…into asynctelepathy
Here is an overview of what got changed by this pull request: Issues
======
+ Solved 6
- Added 25
Complexity increasing per file
==============================
- Assets/Mirror/Tests/Editor/Tcp/TcpTest.cs 3
- Assets/Mirror/Runtime/Transport/Tcp/Server.cs 1
- Assets/Mirror/Runtime/Transport/Telepathy/Client.cs 1
Complexity decreasing per file
==============================
+ Assets/Mirror/Runtime/Transport/Tcp/Client.cs -1
+ Assets/Mirror/Examples/ListServer/ListServer.cs -1
Clones added
============
- Assets/Mirror/Runtime/Transport/Tcp/TcpTransport.cs 1
- Assets/Mirror/Tests/Editor/Tcp/TcpTest.cs 8
- Assets/Mirror/Tests/Editor/Telepathy/TransportTest.cs 4
- Assets/Mirror/Runtime/Transport/TelepathyTransport.cs 1
Clones removed
==============
+ Assets/Mirror/Tests/Editor/FallbackTransportTest.cs -1
+ Assets/Mirror/Tests/Editor/MultiplexTest.cs -1
+ Assets/Mirror/Tests/Editor/FallbackTest.cs -1
See the complete overview on Codacy |
Kudos, SonarCloud Quality Gate passed! 0 Bugs |
# [9.0.0](8.0.1-master...9.0.0-master) (2020-02-08) ### Bug Fixes * don't report error when stopping the server ([c965d4b](c965d4b)) ### Features * awaitable connect ([#55](#55)) ([952e8a4](952e8a4)) ### BREAKING CHANGES * ClientConnect replaced with ClientConnectAsync that can be awaited * fix: TCP transport for async compliance * use async connect * Ignore telepathy tests until they are fixed * It is ok to interrupt sockets * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings
* Asynchronous transport connect * Implement needed interface * feat: you can now await transport connect BREAKING CHANGE: ClientConnect replaced with ClientConnectAsync that can be awaited * fix: TCP transport for async compliance * use async connect * Ignore telepathy tests until they are fixed * It is ok to interrupt sockets * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings
# [9.0.0](8.0.1-master...9.0.0-master) (2020-02-08) ### Bug Fixes * don't report error when stopping the server ([c965d4b](c965d4b)) ### Features * awaitable connect ([#55](#55)) ([952e8a4](952e8a4)) ### BREAKING CHANGES * ClientConnect replaced with ClientConnectAsync that can be awaited * fix: TCP transport for async compliance * use async connect * Ignore telepathy tests until they are fixed * It is ok to interrupt sockets * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings * Remove some warnings
🎉 This PR is included in version 9.0.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
You can now await for the transport client to connect. This makes it a lot easier to capture errors and know when the connection is established
BREAKING CHANGE: ConnectClient has been replaced by ConnectClientAsync