News for hackers. Stuff that matters.
Re:Improving code review time at Meta by nomel on Thursday November 17, 2022 @07:06PM (#33647866)
As someone who has only experienced terrible code reviews, could you give a little summary of what you liked?
Re:Improving code review time at Meta by antiterra on Thursday November 17, 2022 @08:18PM (#33648507)
They probably had some pretty good cat meme image macros.
Re:Improving code review time at Meta by bagels on Thursday November 17, 2022 @08:52PM (#33648800)
Yes good meme macros too. I left a few in my wake.
Re:Improving code review time at Meta by bagels on Thursday November 17, 2022 @08:51PM (#33648797)
Calling you out for lazy work. Teaching you how to use internal tools, company coding standards, suggesting better patterns, digging deep in to the context of system design and maintainability, suggesting other domain experts to tag in the reviews. I learned all sorts of Hack things that obviously I wouldn't learn outside from code reviews.
Re:Improving code review time at Meta by jackblemming on Friday November 18, 2022 @12:49AM (#33650887)
Then why have they produced some of the most garbage software imaginable?
Re:Improving code review time at Meta by dieselgate on Friday November 18, 2022 @01:16AM (#33651027)
Boo, please consider a review of the HN comment guidelines https://news.ycombinator.com/newsguidelines.html

Could you potentially explain what makes it "some of the most garbage software imaginable"?

edit: wording

Re:Improving code review time at Meta by jackblemming on Friday November 18, 2022 @01:28AM (#33651086)
There's already many articles posted to hackernews frequently at the insane internals of the mobile app, to the "meta verse", to many other things. Most of which are posted by ex-Facebook engineers. I don't need to rehash the same conversations, nor do I need to grant Facebook of all corporations any sort of courtesy.
Re:Improving code review time at Meta by gtirloni on Friday November 18, 2022 @04:09AM (#33652197)
The courtesy is towards us, people engaging in this discussion. Meta is not a party here.
Re:Improving code review time at Meta by RussianCow on Friday November 18, 2022 @01:51AM (#33651248)
Name one successful company that only produces beautiful, elegant, and readable code, with no nasty warts or hacks.

I'll wait.

Re:Improving code review time at Meta by KronisLV on Friday November 18, 2022 @06:30AM (#33653185)
> Name one successful company that only produces beautiful, elegant, and readable code, with no nasty warts or hacks.

Honestly, I'd genuinely like to hear some of the answers to this, or even just mentions of good codebases to explore.

That said, I don't think that there will be that many that satisfy all of the requirements listed here, because as the scope of a project grows, there will almost inevitably be some technical debt or accidental complexity, or even subjective things.

Re:Improving code review time at Meta by bagels on Friday November 18, 2022 @02:09AM (#33651363)
I mean, I generally think comments with this kind of attitude aren't looking for serious responses, so I normally don't respond.

That said, there are three things to consider: 1) I honestly don't care whether you think it's garbage or not. 2) I saw some amazing software there, and I saw some that wasn't amazing. 3) It's an immense business, and there is a certain amount of inherent dysfunction that comes with that many people working on anything together much as you would see at any large company.

Re:Improving code review time at Meta by dieselgate on Friday November 18, 2022 @01:20AM (#33651053)
How are small things like "you left a commented out print/log" or "use color var not hex code" handled - or that a 'company coding standards' type thing. Are those types of things potentially automated with a linter or similar?