簡化工作流之程式碼審查回覆訊息生成
阿新 • • 發佈:2021-01-31
在一個規範化的研發流程中,一般遵循如下流程:
1. 開發階段:研發功能或者修復bug,在本地自測。
1. 程式碼稽核階段:提交程式碼,並請求團隊內人員做code review。
1. 測試環境測試階段:部署到測試環境並請求測試。
1. 釋出線上待測階段:測試環境通過測試釋出到線上進行測試。
1. 驗收完成任務:線上驗證成功,關閉這個任務。
實際上這只是一種最理想化的過程,因為我們預設每次狀態流轉都是順利的,開發沒有毛病,測試一次就通過,現實中的研發流程的情況更復雜,如圖1所示。
![圖1 研發流程圖](https://img2020.cnblogs.com/blog/782095/202101/782095-20210131200727207-566795894.png)
圖1
整個過程一氣呵成,環環相扣。而其中可以被自動化的正好是第二步:請求他人進行code review的時候的反饋訊息。
根據實踐的經驗,比較好的內容格式如下(包含Markdown格式,因為跟蹤任務的系統支援這種格式):
```
**Changes has been committed to feature/xxx-xxx**
- https://git.xxx.com/xxxx/ddaf18f9be4613c31363d4c92b8bafc3sdfdsf
**Details**
Remove invalid logic for admin pannel
```
由於每次走到Code Review的步驟的時候都需要寫類似的回覆在任務管理系統中,所以考慮使用Python指令碼去自動生成這段文字,簡化工作。
根據樣例回覆進行分析,需要獲取專案的分支名(任務目標分支),專案最後一次提交的commit id去組裝第二行的git commit的連結,然後Details的內容可以從git log中的提交資訊裡面提取。
第一步:獲取分支名稱。
為了簡化過程,預設專案的當前分支就是我們需要的分支,那麼問題簡化為獲取當前分支名。可以利用git的相關命令實現,如下:
```
git branch | sed -n '/\* /s///p'
```
第二步:獲取commit id。
而獲取commit id也非常簡單,只需要如下命令:
```
git rev-parse HEAD
```
第三步:獲取提交資訊。
還需要獲取提交資訊,利用git log的命令進行過濾也能得到:
```
git log --pretty=format:"%s" -1
```
git log --pretty=format命令很強大,除了獲得提交資訊外,還有如下引數可以使用。
```
%H 提交物件(commit)的完整雜湊字串
%h 提交物件的簡短雜湊字串
%T 樹物件(tree)的完整雜湊字串
%t 樹物件的簡短雜湊字串
%P 父物件(parent)的完整雜湊字串
%p 父物件的簡短雜湊字串
%an 作者(author)的名字
%ae 作者的電子郵件地址
%ad 作者修訂日期(可以用 -date= 選項定製格式)
%ar 作者修訂日期,按多久以前的方式顯示
%cn 提交者(committer)的名字
%ce 提交者的電子郵件地址
%cd 提交日期
%cr 提交日期,按多久以前的方式顯示
%s 提交說明
```
所以第二步也可以使用git log命令實現,如下所示:
```
git log --pretty=format:"%H" -1
```
當然還需要在後面加一點人性化的感謝的話,畢竟是麻煩其他人來對你程式碼進行稽核,說一些感謝的話吧,這裡我就用一個list來裝一些感謝的話,然後隨機獲取一段貼到最後。
如果是以面向過程的方式去編寫,那麼可以編寫如下程式碼:
```
#coding=utf-8
#!/usr/bin/python
import os, subprocess
import random
# use subprocess to get the current branch name from output
def get_branch_name(cd_path):
os.chdir(cd_path)
status, branch_name = subprocess.getstatusoutput("git branch | sed -n '/\* /s///p'")
# print(output)
# exit(0)
return branch_name
def get_latest_git_log(cd_path):
"""
docstring
"""
os.chdir(cd_path)
status, log_info = subprocess.getstatusoutput("git log --pretty=format:\"%s\" -1")
return log_info
def get_latest_commit_id(cd_path):
os.chdir(cd_path)
status, commit_id = subprocess.getstatusoutput("git rev-parse HEAD")
return commit_id
def get_reviewer_by_random(reviewers):
return random.choice(reviewers)
def get_thanks_words_by_random(thanks_words):
return random.choice(thanks_words)
def create_comment(reviewers, branch_name, log_info, commit_id, thanks_words):
print(get_reviewer_by_random(reviewers))
print("*Changes made has been committed to " + branch_name + "*")
print("- https://git.xxxxx.com/someproject/subname/-/commit/" + commit_id)
print("*Details*")
print("-" + log_info)
print(get_thanks_words_by_random(thanks_words))
branch_name = get_branch_name('/Users/tony/www/autoWork')
log_info = get_latest_git_log('/Users/tony/www/autoWork')
commit_id = get_latest_commit_id('/Users/tony/www/autoWork')
reviewers = [
'[~Harry]',
'[~Tom]'
]
random_thanks_words = [
'Review it please, thanks.',
'Actually, I am glad to see you have time to review it, thanks a lot.',
'Please check it if you have free time, thanks.',
'Check it please.'
'Waiting for your code review, thank you.'
]
create_comment(reviewers, branch_name, log_info, commit_id, random_thanks_words)
```
由於Python指令碼和專案沒有放在一個目錄下面,所以每次在執行git相關命令之前都需要先cd到目標專案目錄下。而分別執行git命令的時候使用subprocess.getstatusoutput()來執行,方便獲取標準化輸出的結果。這裡之所以不使用os.system來執行命令,是因為os.system執行命令的返回值裡面包括兩個部分,第一部分是命令的結果輸出,第二部分是結果是否成功的識別符號。
例如執行os.system("git branch | sed -n '/\* /s///p'")會返回如下內容:
```
feature/ST-247
0
```
第一行是我們獲取到的分支名,第二行是成功的識別符號,0表示命令沒有任何問題。
所以我考慮使用subprocess.getstatusoutput來執行命令,這個函式會分別返回結果標識和輸出,方便得到想要的執行輸出結果。
雖然程式碼還可以進一步優化,但是已經能滿足我的需求了,執行這個指令碼就能得到如下的輸出結果:
```
[~Harry]
*Changes made has been committed to feature/ST-247*
- https://git.xxxxx.com/someproject/subname/-/commit/d21033057677e6d49d9cea07c64c49e35529545dx
*Details*
- Remove some invalid logic
Please check it if you have free time, thanks.
```
如果改寫成面向物件的方式會更好,呼叫更簡單,傳遞引數也更少,採用Python3語法編寫的程式碼如下所示:
```
#coding=utf-8
#!/usr/bin/python
import os
import subprocess
import random
class CommitComment:
def __init__(self, project_path: str, reviewers: list, thanks_words: list):
self.project_path = project_path
self.reviewers = reviewers
self.thanks_words = thanks_words
# use subprocess to get the current branch name from output
def get_branch_name(self) -> str:
os.chdir(self.project_path)
status, branch_name = subprocess.getstatusoutput("git branch | sed -n '/\* /s///p'")
return branch_name
# use subprocess to get the latest commit message from git log
def get_latest_git_log(self) -> str:
os.chdir(self.project_path)
status, log_info = subprocess.getstatusoutput("git log --pretty=format:\"%s\" -1")
return log_info
# use subprocess to get the latest commit id from git log
def get_latest_commit_id(self) -> str:
os.chdir(self.project_path)
status, commit_id = subprocess.getstatusoutput("git rev-parse HEAD")
return commit_id
def get_reviewer_by_random(self) -> str:
return random.choice(self.reviewers)
def get_thanks_words_by_random(self) -> str:
return random.choice(self.thanks_words)
def create_comment(self):
print(self.get_reviewer_by_random())
print("*Changes has been committed to " + self.get_branch_name() + "*")
print("- https://git.xxxx.com/MyProject/ProjectName/-/commit/" + self.get_latest_commit_id())
print("*Details*")
print("-" + self.get_latest_git_log())
print(self.get_thanks_words_by_random())
thanks_words = [
'Review it please, thanks.',
'Actually, I am glad to see you have time to review it, thanks a lot.',
'Please check it if you have free time, thanks.',
'Check it please.'
'Waiting for your code review, thank you.'
]
reviewers = [
'[~Harry]',
'[~Tom]'
]
comment = CommitComment('/Users/tony/www/autoWork', reviewers, thanks_words)
comment.create_comment() # will print out the complete comment
```
thanks_words列表可以在增加多一點,這樣隨機獲取之下重複的概率會更少。當然最後一段也可以自己每次diy,畢竟感謝要發自內心的最好。
這種簡化工作流的指令碼本質是減少重複性勞動,特別是一天完成了很多個任務的時候。但是**反思**本身是無法被**簡化**的,不做工作的奴隸,而是工作的主人。
拋磚引玉,希望對自己和未來的自己也是一個還原映象。
Todo:
1.可以每天定時執行這個指令碼去生成回覆訊息。
2.通過指令碼傳參來動態選擇需要被處理的專案目錄。在這個案例程式碼中是hard code的,預設是選擇了autoWork這個專案。
3.還可以考慮接入語料庫(thanks words),這樣感謝的話永不重複,還能學點新單詞