Tim Abbott

Results 1303 comments of Tim Abbott
trafficstars

Merged, thanks for doing this @jai2201! Hopefully you've learned a bit about how to do simple but efficient data structures through this project :)

@abhijeetkaur @showell what are the next steps on this PR?

This looks to me to be a correct implementation of fixing this inconsistency by adding these groups to the system bot realm. I feel like it might be more correct...

This is great, merged with a few changes: * The tweaks mentioned here: https://github.com/zulip/zulip/pull/22352#discussion_r941909110 * Made sure the comment for the migration appears at the top of the file; that's...

Yeah, that's a possibly surprising interaction with how GitHub's "merge once CI passes" feature works -- basically it'll merge the next time the PR passes CI, regardless of whether there...

@adnanislam can you clean up your commit history to match our style? Check out our [GitHub guide](https://zulip.readthedocs.io/en/latest/git-guide.html) and [commit guidelines](https://zulip.readthedocs.io/en/latest/contributing/version-control.html) for more details.

I merged #10290, after grabbing a line of test code from #9967. Thanks for your work on this @skunkmb! You may want to chat with @lonerz about which of you...

Reopening, since this reverted (see #10290 for discussion): 62bee26ea71cb0f54cb32620a15c69d66c344c68