-
Notifications
You must be signed in to change notification settings - Fork 570
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
7.0 release checklist / misc items #1413
Comments
@stebet what do you think of this item?
|
All for it! |
Does we need QueueDeclarePassiveAsync too? |
The
|
Part of #1413 * Prefer explicit types instead of `var`
* Various editor suggestions Part of #1413 * Prefer explicit types instead of `var` * Hopefully eliminate this test flake. * Publish smaller messages to make timeouts less likely.
readonly
fields instead ofget;
TODO
comments in the codeITcpClient
or come up with a better way of setting socket options. See Removed ReceiveBufferSize and SendBufferSize to improve message rates #1415 and this discussion. Note:TcpClientAdapter
was made public in Make TcpClientAdapter public #1417var
with explicit types when appropriate.await
whenIsCompletedSuccessfully
istrue
(though, frankly, I prefer easier-to-read code 😸). These appear to be relevant:QueueDeclarePassive
have async versions.The text was updated successfully, but these errors were encountered: