>It seems to mostly be a me problem, on reviews of other coworkers PRs her comment seem reasonable. That could just be not fully understanding scope or task at hand.
What do you think explains the difference?
* Are there ways you can explain yourself better that others are doing?
* Could the problem honestly be with your code being hard to understand? Could you simplify?
* Could you deliver in smaller chunks, that are each easier to understand?
* Is it about working on a tech or product area that they aren't so familiar with? Perhaps you can help bridge that gap with a lightning talk or some new documentation?
>I have not had a direct conversation as I'm not sure how to go about it or phrase it. We seem to have a good relationship and I don't want to potentially ruin it going about this in the wrong way.
You've got to place trust in your colleagues that you can have adult conversations about what's working well and what's not. It doesn't have to be a big deal!
It could be as simple as:
"Hey <name>, I feel like a few times lately PR reviews hasn't been smooth. Have you noticed that too? Is there anything you think I could do differently?"
>We have basic standards
One thing that could help here is improving your team standards for a PR author! E.g. at my company, we have a PR template which encourages you to add a screenshot/screencast of your feature working.
We also encourage adding test steps to our PR template, to make it easy for reviewers to try the thing themselves. Good test steps might look like:
* Run the app as normal
* Turn on feature flag XYZ
* Head to localhost:3000/new-page and click all the widgets
* You should see the widgets created in your db with this query:
It sounds like work, but it's probably the steps you did anyway before you put the PR up! so it's more of a mindset thing