FiniteLooper@lemm.eetoProgramming@programming.dev•In Rust we trust? White House Office urges memory safety - Stack OverflowEnglish
3·
10 months agoinclude $pixels;
include $pixels;
This is why I bring my laptop into the shower
“They make me write PHP”
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.