Comic Code Reviews
44 points
6 days ago
| 11 comments
| jona.ca
| HN
halflife
8 minutes ago
[-]
If it’s an addendum then fine. But can’t replace textual review which is much easier to parse, at least for me.

BTW, amazing they chose a review that exemplifies why hooks are a horrible horrible mistake for public API.

reply
JonathanAquino
6 days ago
[-]
I’ve been experimenting with a way to make code reviews more understandable - turning tricky pull requests into short comic strips.

The blog post shows an example generated from a real PR: summarizing the changes, anthropomorphizing the components, and making the flow visually obvious. It’s meant to help reviewers grasp intent quickly and make reviews a bit more fun.

Curious whether others have tried visual or narrative aids in their review process, and whether this could be practical for real teams.

reply
treetalker
4 hours ago
[-]
This could be a fun way to educate the judge about why my opposing counsel's position is laughably wrong.

Sadly, the fun would end with a reprimand or sanctions order. Cf. https://news.ycombinator.com/item?id=43866303 ("Don't watermark your legal PDFs with purple dragons in suits").

Might work for bringing associate attorneys up to speed in a new case, or for teaching concepts to law students, though!

reply
justinclift
1 hour ago
[-]
Using visual approaches, including comics, is a reasonable idea I've been looking into personally as well (mostly using style transfer from manga). :)

But, the actual concepts communicated need to be clear. In your example strip here, it doesn't seem to be meeting that bar for a reviewer. :(

Keep at it though, as I get the feeling this is the kind of thing that will work after a few important "aha!" ideas and tweaks happen to the generating process. :)

reply
klooney
1 hour ago
[-]
I'm not against it, but this particular strip seemed a little incoherent. It might need manual storyboarding.
reply
guitarbill
2 hours ago
[-]
I really like the idea... but I have to admit my first visceral reaction was "I hate this". I think it's because the tone and style is quite infantile/childish. A good experiment nonetheless. Maybe there's a middle ground somewhere?
reply
shermantanktop
2 hours ago
[-]
What’s your line for “childish“?

I’ve been guilty of injecting bits of whimsy, sarcasm, and other unserious behaviors in a corporate We Mean Business environment.

I’ve toned it down because I recognize some people are very resistant to seeing both the serious and humorous aspects of a situation at the same time.

reply
ChrisMarshallNY
3 hours ago
[-]
I think it’s a fun idea.

Not sure if it would be used, though. Being on HN front page helps.

It would also have to contain a lot of content, and be indexed well.

reply
namanyayg
2 hours ago
[-]
Interesting idea but unfortunately the given example comic makes very little sense.

It was difficult to parse even as someone who's familiar with these concepts, and I think it will hurt more than help any newbies.

reply
continuational
9 minutes ago
[-]
I think this kind of slop has negative value. It's unclear how much of the information in the comic is hallucinated, and the malformed code "for i = i1+|>" and nonsense text "(starts write hooks!" doesn't bode well.
reply
Paracompact
1 hour ago
[-]
Exactly my thought. Can anyone here say otherwise?
reply
collingreen
1 hour ago
[-]
I didn't have any trouble with the comic but I do already know these things about how react hooks work so I'm not going in fresh.

I think it's a little whimsical, perhaps too much for what info it conveys (a bullet list with the same component names would probably be equally informative), but I thought it was easy to understand and follow. I think there is -something- here; I don't need THIS comic but if it was more about the context and goals of the change then maybe that would be powerful. Especially if it was consistently done over many PRs.

reply
SoftTalker
1 hour ago
[-]
We are well and truly doomed. Why not just make it a TikTok.
reply
roywiggins
36 minutes ago
[-]
reply
lukebechtel
4 minutes ago
[-]
ok fine I'll add another github workflow...
reply
keithnz
1 hour ago
[-]
not sure about for pull requests, but for protocols it could be interesting.

I asked it to generate a comic for https negotiation over tcp https://imgur.com/a/0p0Pzum I think with a bit more prodding it might be interesting for documenting protocols

reply
joshdavham
4 hours ago
[-]
You actually might be on to something... AI aside, it's often a good idea to include visuals in a PR such as diagrams.

But having something like a comic where it's both visual and communicative in a more conversational/narrative way could prove pretty effective. Also if you can throw some humour in there, it could potentially add even more comprehensibility, etc.

Thanks for sharing!

reply
tommica
59 minutes ago
[-]
Fun little comic :) wish it had more about WHY the rules exist, but I assume that would be hard to squish into a panel
reply
antonvs
4 hours ago
[-]
It seems to me the level this comic is at is such that anyone who needs an aid like this would not be capable of providing a meaningful review on the pull request.

If the goal is to encourage rubber-stamping by bystanders, it might help.

reply
BurningFrog
3 hours ago
[-]
It's valuable to make a difficult task easier, even for those who could do the task without the help.
reply
antonvs
2 hours ago
[-]
I'd rather just have a high-level summary in English for that.

This seems to take dumbing-down beyond any sensible level.

reply
hyperhello
2 hours ago
[-]
People who dumb down seem to think it's easier to be dumb. Not everyone seems to agree, fortunately.
reply
qwertytyyuu
3 hours ago
[-]
Looks like a fun inclusion
reply
tantalor
2 hours ago
[-]
Any code review complicated enough to benefit from this should be split up into smaller units for review.
reply