Asked a senior engineer to roast my PR style instead of just asking if it 'looked okay' and finally got useful feedback

Kept asking 'does this look fine' in code review and kept getting 'looks good' back, which taught me nothing. Switched to explicitly asking someone senior to be harsh about naming, structure, whatever bugged them. First session was humbling, apparently I over-comment and under-name variables. Second session two weeks later was actually a short conversation instead of a teardown.