I made a branch, make commits, and then make a PR. I don’t care about the number of commits because sometimes a reviewer might be able to make more sense of a PR if they view each commit instead of all the changes at once.
For us we just make sure that the branch builds and passes tests before merging it in, and just do a general look over to make sure everything looks correct, follows best practices, etc. if the UI was changed I usually add screenshots of before/after or a screen recording of me using the feature. Sometimes these can really help a reviewer understand what all the changes mean.
It’s really accelerated in the past few years. It’s nearly impossible to just read an article or use any product without giving it some kind of information. Lots of people (myself included many times unfortunately) just accept this. I mean, what can be done? If you want or need to use the thing you almost have no choice. If you want to avoid information leaks or being tracked you have to do so much research and work just to find an option, and then hope they don’t get purchased by a company that will reverse it all. I hate it.
So what is a good camera system to own? I currently have ADT and I’m really not happy with it. It’s expensive and the cameras only record 30 second clips. It can detect motion, it records 30 seconds and that’s it, regardless of how long the motion event actually takes.
Example: someone drops off a package and they hang out on my porch - I have no idea what happens after the 30 second mark! It’s insane. No way to change this either. The only option is for how long to wait between 30 second clips, and the lowest option is 2 minutes.
This same advice is true for the JavaScript situation as well sometimes