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
Release v0.11.2 #698
Release v0.11.2 #698
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm other than nits.
You mentioned there's a fix for #687, is that actually in here?
config.sample.yaml
Outdated
# This is used to stem the flow of requests in case of a mass quit/leave, which might | ||
# slow down the homeserver. | ||
leaveConcurrency: 10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no mention in the changelog?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch, this is a develop feature atm.
@@ -1,6 +1,11 @@ | |||
"$schema": "http://json-schema.org/draft-04/schema#" | |||
type: "object" | |||
properties: | |||
advanced: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what about ircHandler->leaveConcurrency?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above.
if (bridgedClient.isDead()) { | ||
return; // We don't want to include dead ones. | ||
if (!bridgedClient || bridgedClient.isDead()) { | ||
// We don't want to include dead ones, or ones that don't exist. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
var isNonexistentOrDead = !bridgedClient || bridgedClient.isDead()
if (isNonexistentOrDead) {
same amount of lines, expresses semantics in code instead of comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I'll include this in the next release since it's non-critical (and saves us having to apply it to two branches) but definitely reads better.
Should be fixed by the null checking, so I'll add a statement to that effect in the changelogs. |
That didn't actually end up in the changelog now, did it? |
Oops, my bad. Sorry :( |
No description provided.