code review 黑话摘录

code review 黑话摘录

最近在给 element plus 提交 pr 的时候看到一些莫名其妙的单词缩写。查阅后发现其是有功能的,现摘录如下。

以后在写代码 review 的时候,可以用一下,逼格拉满。

简写含义
LGTMLooks good to me,表示认可这次 PR,同意 merge 合并代码到远程仓库
NACK/NAKnegative acknowledgement, i.e. disagree with change and/or concept
ASAPas soon as possible! 尽快
ACKacknowledgement, i.e. agreed/accepted change
RFCrequest for comments, i.e. I think this is a good idea, lets discuss
WIPWork In Progress 「进展中」,常见词汇,这里做为 Best Practice 单独提出来,主要针对改动较多的 PR,可以先提交部分,标题或 Tag 加上 WIP,表示尚未完成,这样别人可以先 review 已提交的部分
IIRC: if I recall correctly
IANAL:“ I am not a lawyer ”, but I smell licensing issues
IMO:(In my opinion),在我看来
TL;DR: Too Long; Didn't Read 「太长懒得看」,README 文档常见。
AFAIK/AFAICTas far as I know / can tell
PRPull request「合并请求」
CRCode Review 「代码审查」
PTALPlease Take A Look. 你来瞅瞅?用来提示别人来看一下
TBRTo Be Reviewed. 提示维护者进行 review
TBDTo Be Done(or Defined/Discussed/Decided/Determined). 根据语境不同意义有所区别,但一般都是还没搞定的意思。
TBHTo be honest 「老实说」
ATMat the moment 「现阶段」,
Nuxt3 useAsyncData vs useFetch
Nuxt3 请求数据

评论区

评论加载中...