minus-squaresteventhedev@lemmy.worldtoProgrammer Humor@programming.dev•Better merge it, too!linkfedilinkarrow-up34·1 day agoI have either written or gotten a variant of every single one of these comments 🫠: Please include the JIRA task in the commit title. Did you run any manual testing? Where’s the PRD link in the commit message? Can you please split this into multiple smaller commits? Can you combine these two commits? Did you email Jon about this because he’s working on that project with Sarah and you might be duplicating efforts. This should be named BarFoo instead of FooBar. Why aren’t you using CorporateInternalLib16 that does 90% of this? Why aren’t you using ThirdPartyPaidLibByExEmployee? Why aren’t you using StandardLib thing you forgot existed? All our I/O should be async. All our hot loop code needs to be sync. This will increase latency of NonCoreBusinessFlow by 0.01%. can you shave some time off so we can push in feature B also? Please add a feature flag so we can do gradual rollout. What operational levers does this have? Lgtm - just address those comments linkfedilink
minus-squaresteventhedev@lemmy.worldtoScience Memes@mander.xyz•ExcellinkfedilinkEnglisharrow-up33·4 days agoAll images are stacked bar charts if you try hard enough linkfedilink
minus-squaresteventhedev@lemmy.worldtoToday I Learned@lemmy.world•TIL That each missile fired by Israels Iron Dome cost anywhere between 40k to 50k US per missilelinkfedilinkEnglisharrow-up1arrow-down1·16 days agoSo you hope that Hamas and Hezbollah continue firing rockets at civilians? linkfedilink
I have either written or gotten a variant of every single one of these comments 🫠:
Please include the JIRA task in the commit title.
Did you run any manual testing?
Where’s the PRD link in the commit message?
Can you please split this into multiple smaller commits?
Can you combine these two commits?
Did you email Jon about this because he’s working on that project with Sarah and you might be duplicating efforts.
This should be named BarFoo instead of FooBar.
Why aren’t you using CorporateInternalLib16 that does 90% of this?
Why aren’t you using ThirdPartyPaidLibByExEmployee?
Why aren’t you using StandardLib thing you forgot existed?
All our I/O should be async.
All our hot loop code needs to be sync.
This will increase latency of NonCoreBusinessFlow by 0.01%. can you shave some time off so we can push in feature B also?
Please add a feature flag so we can do gradual rollout.
What operational levers does this have?
Lgtm - just address those comments