You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Sorry for the silly question, but I can't figure this acronym.
At work, we use "LGTM" for "Looks good to me", when approving a pull request. So, I assume it's something close, but I can't figure out what that first "T" is.
The text was updated successfully, but these errors were encountered:
Good question, I actually don't have it spelled out in the readme. It means "looks testable to me" and the rationale is described here. Basically, our tests take about 20mins, so we want to make sure the human reviewer is happy with the code before it gets tested.
Sorry for the silly question, but I can't figure this acronym.
At work, we use "LGTM" for "Looks good to me", when approving a pull request. So, I assume it's something close, but I can't figure out what that first "T" is.
The text was updated successfully, but these errors were encountered: