3 ms·
I thought it was "Let's Get This Merged" and commented it on my own PR once.
by CameronNemo 4y ago
I thought it was "Let's Get This Merged" and commented it on my own PR once.
- david_van_loon 4y agoIt's an ambiguous acronym. I've used it both ways.
- Thorrez 4y agoAll the front page Google results appear to say Looks Good To Me https://www.dictionary.com/e/acronyms/lgtm/ https://www.dictionary.com/e/acronyms/lgtm/ https://www.urbandictionary.com/define.php?term=LGTM https://www.urbandictionary.com/define.php?term=LGTM https://www.freecodecamp.org/news/what-do-cryptic-github-comments-mean-9c1912bcc0a4/ https://www.freecodecamp.org/news/what-do-cryptic-github-com... https://knowyourmeme.com/memes/lgtm https://knowyourmeme.com/memes/lgtm https://www.cyberdefinitions.com/definitions/LGTM.html https://www.cyberdefinitions.com/definitions/LGTM.html Urban Dictionary does have Let's Get This Merged as its second definition, but it's at a net of -121 points, whereas Looks Good To Me is at net 941, and was added 10 years earlier. That doesn't seem ambiguous to me.
- rob74 4y agoWell, I assume you wouldn't submit a PR if it didn't look good to you, so the acronym is appropriate...
- dathinab 4y agoyou have no idea (1) what some people do when some external force tires to make them contribute to opensource ... 1: ok, you probably do know as most people on HN probably do know
- steinuil 4y agoI thought it meant "Let's Get This Merge" like it was a play on "let's get this bread". I know it stands for "Looks Good To Me" now but I still prefer my reading :)