[Twisted-Python] reviews
![](https://secure.gravatar.com/avatar/e1554622707bedd9202884900430b838.jpg?s=120&d=mm&r=g)
Hello Twisted friends, This is just a friendly reminder that https://twisted.reviews <https://twisted.reviews/> is full to bursting - 34 open reviews right now, some dating back as far as May of last year. If you've got a few free minutes, can you find a ticket on that list and review the associated PR? (I recently fixed the report so it's actually in the order you should review them...) A short review queue is a happy review queue. Remember also that this isn't for nothing, if you do code reviews on Twisted you get internet points and bragging rights, at https://twistedmatrix.com/highscores/ <https://twistedmatrix.com/highscores/>. This isn't just for project members; if you did not already know, the rule is that if you're a project member you can review anything, but if you're an external contributor, you can review submissions by project members (and it's up to them to decide if your review is adequate before acting on it). Project members (should) have a "*" next to their name in the "submitted by" list. (If they don't I think someone needs to go manually update a list of strings, feel free to report issues if that's wrong.) Happy hacking, -glyph
![](https://secure.gravatar.com/avatar/cf223b7cf77583c0a2665bad01f84f11.jpg?s=120&d=mm&r=g)
On Tuesday, 20 July 2021 05:38:23 BST Glyph wrote:
Hello Twisted friends,
This is just a friendly reminder that https://twisted.reviews <https://twisted.reviews/> is full to bursting - 34 open reviews right now, some dating back as far as May of last year. If you've got a few free minutes, can you find a ticket on that list and review the associated PR? (I recently fixed the report so it's actually in the order you should review them...) A short review queue is a happy review queue.
$ curl https://twistedmatrix.com/trac/report/25 curl: (7) Failed to connect to twistedmatrix.com port 443: Connection refused
![](https://secure.gravatar.com/avatar/e1554622707bedd9202884900430b838.jpg?s=120&d=mm&r=g)
If anyone has the time to deal with it, we *do* have a free subscription donated by the friendly folks over at Sentry; if we set up the integration in all of our services it might lead to the discovery of some interesting bugs. Let me know if you need credentials or an invite or something. -g
![](https://secure.gravatar.com/avatar/e1554622707bedd9202884900430b838.jpg?s=120&d=mm&r=g)
Update: I went through the queue and gardened it substantially. There were a bunch of tickets where folks had forgotten to remove the review keyword upon closure, some with malformatted commit messages so they weren't closed (this is very hard to enforce mechanically, but please, use the template in the PR boilerplate), and some with merge conflicts that needed addressing before review. I also pushed the "update branch" button on a whole lot of PRs so hopefully some CI detritus that made tests fail for spurious reasons will be cleared out, and everything will be reasonably up to date. We are now down to 17 matches, but this does give me a commanding lead on this month's leaderboard: I look forward to being dethroned. -g
![](https://secure.gravatar.com/avatar/cf223b7cf77583c0a2665bad01f84f11.jpg?s=120&d=mm&r=g)
On Tuesday, 20 July 2021 05:38:23 BST Glyph wrote:
Hello Twisted friends,
This is just a friendly reminder that https://twisted.reviews <https://twisted.reviews/> is full to bursting - 34 open reviews right now, some dating back as far as May of last year. If you've got a few free minutes, can you find a ticket on that list and review the associated PR? (I recently fixed the report so it's actually in the order you should review them...) A short review queue is a happy review queue.
$ curl https://twistedmatrix.com/trac/report/25 curl: (7) Failed to connect to twistedmatrix.com port 443: Connection refused
![](https://secure.gravatar.com/avatar/e1554622707bedd9202884900430b838.jpg?s=120&d=mm&r=g)
If anyone has the time to deal with it, we *do* have a free subscription donated by the friendly folks over at Sentry; if we set up the integration in all of our services it might lead to the discovery of some interesting bugs. Let me know if you need credentials or an invite or something. -g
![](https://secure.gravatar.com/avatar/e1554622707bedd9202884900430b838.jpg?s=120&d=mm&r=g)
Update: I went through the queue and gardened it substantially. There were a bunch of tickets where folks had forgotten to remove the review keyword upon closure, some with malformatted commit messages so they weren't closed (this is very hard to enforce mechanically, but please, use the template in the PR boilerplate), and some with merge conflicts that needed addressing before review. I also pushed the "update branch" button on a whole lot of PRs so hopefully some CI detritus that made tests fail for spurious reasons will be cleared out, and everything will be reasonably up to date. We are now down to 17 matches, but this does give me a commanding lead on this month's leaderboard: I look forward to being dethroned. -g
participants (3)
-
Adi Roiban
-
Barry Scott
-
Glyph