and I’d also say that I focus on the things I don’t understand. If I don’t understand it, it can mean I’m stupid and missed something. In this case I learn. It can also mean that it lacks documentation or is simply a bug. When you’re reading this now, there’s also a big chance other developers have this issue when they read the code later on. After it’s merged, you’re less likely to change it.
Reviewing is a process of collaboration. You learn from each other. Don’t worry too much about asking stupid questions because most questions aren’t stupid.
AFAIK this is only true for installer PRs. Testing PRs using RPM packages is still open.