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:
-
Show this thread
-
Oh poo I messed up the thread. Fixing...
1 reply 0 retweets 0 likesShow this thread -
- 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
1 reply 0 retweets 0 likesShow this thread -
- I looked at what the code used to look like before vs new proposal, found a difference
1 reply 0 retweets 0 likesShow 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
1 reply 0 retweets 0 likesShow this thread -
- So I knew the new PR probably wouldn't work for 1/2 places this code is called
1 reply 0 retweets 0 likesShow this thread -
How can I teach that, other than giving people time and opportunity to become familiar w/the whole codebase & its history...? idk.
2 replies 0 retweets 0 likesShow this thread -
Replying to @Carols10cents
Not quite sure I’m totally following, but it sounds like you could write a test that would have failed for the PR? If tests are comprehensive, you don’t necessarily need to know the history.
3 replies 0 retweets 5 likes
I would add to this: never too late! Add the test now that you saw the edge case get forgotten.
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.