Ok so here's an example of the context I have on a codebase that I can't figure out how to efficiently transfer to other reviewers:
-
-
- PR comes in to move A to B - I remember that a while ago (sept), we moved B to A, so I knew to go look for that to remember why
Show this thread -
- I looked at what the code used to look like before vs new proposal, found a difference
Show this thread -
- That difference is important to one of the two places this code is called, neither of those places was changed in the PR
Show this thread -
- So I knew the new PR probably wouldn't work for 1/2 places this code is called
Show this thread -
How can I teach that, other than giving people time and opportunity to become familiar w/the whole codebase & its history...? idk.
Show this thread
End of conversation
New conversation -
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.