This message was deleted.
# feature-requests
s
This message was deleted.
b
I love this idea. I agree that the majority of the time Graphite should surface positive messages. As of now, it's technically pretty neutral phrasing, but it lands for many people as not neutral, especially if they're insecure for any reason (new hire, having a hard time, etc) One framing I've found helpful interpersonally (and it applies here too) is to phrase things in ways where you aren't talking about the person. So instead of "You reviewed more PRs than average this week", something like "This was a lighter week than usual". It's about the week, not about the person. This is actually more accurate too! Maybe you had a bunch of meetings that week, or an offsite, or had to sit on hiring panels, or your tasks were smaller/bigger, or you were sick, or religious holidays, or Github was down so nobody got things done, or, or, or... Most of the time, these fluctuations aren't due to the person, they're due to the circumstances the person finds themselves in. So some sample copy from me (not a copy editor) • 🪶 It was a lighter week than usual • šŸ”„ Wow, what a week for merging! • šŸ” Hey Sherlock, this was a big week for reviews! • āœļø Wow so much, very code!
e
to clarify, it's not about the phrasing, it's about showing how you are "performing" in terms of arbitrary metrics with respect to team average: by design that means that half the team will be told they are below average with respect to the chosen metrics.
šŸ‘ 1
we would rather have the option to hide this altogether and decide for ourselves what metrics we value (and whether we want to constantly remind team members about their score on those metrics)
p
Hey guys! Thanks for the helpful feedback as always - heard on all points here. I'll bring this up with the team!
šŸ™Œ 1
@Ben Jaffe u are a good copy editor