This message was deleted.
# feature-requestss
Slackbot
10/16/2023, 6:55 PMThis message was deleted.
b
Ben Jaffe
10/16/2023, 8:01 PMI love this idea -- to add to it, if there was a way to visually see that bigger PRs lead to fewer comments per LOC, that'd be a good argument for smaller PRs. Maybe a graph with LOC on the Y axis and # comments per LOC on the X axis.
❤️ 2
k
kush
10/16/2023, 9:26 PM@Raimundo Perez great idea! i’ll pass this along 🙂
❤️ 1
kush
10/16/2023, 9:26 PM@Ben Jaffe funny you mention that — we actually wrote a blog post on this topic not too long ago!
https://graphite.dev/blog/the-ideal-pr-is-50-lines-long#average-number-of-inline-comments-by-pr-size
b
Ben Jaffe
10/16/2023, 9:29 PMThat's exactly what inspired me to want this!
Ben Jaffe
10/16/2023, 9:29 PMAnd I think the chart per-LOC would be really neat
k
kush
10/16/2023, 9:30 PMlove it! cc @Aryaman for viz
👍 1
b
Ben Jaffe
10/16/2023, 9:30 PMBut don't let me hijack the original thread's request, cause I think that's more important 😄
2 Views