search

Ptt-official-app / go-bbs

BBS file database manager system library in Golang.
Apache License 2.0
80 stars 43 forks source link

Feature/#78 取得使用者設定 #85

Closed AmberFu closed 3 years ago

AmberFu commented 3 years ago

👏 解決掉的 issue / Resolved Issues

⛏ 變更內容 / Details of Changes

y2468101216 commented 3 years ago

請先修好 ci

PichuChen commented 3 years ago

@AmberFu 請問一下 CI 的問題你那邊有辦法處理嗎？

PichuChen commented 3 years ago

@AmberFu 變數的部分還是有些 CI 上的問題喔

AmberFu commented 3 years ago

@PichuChen 不好意思，我對 Golang 的 naming 不太熟悉，希望這次都更正到了！ 想請問一下是需要您那邊幫忙跑 CI 測試還是 commit 一段時間就會自動檢測呢？ 有點好奇，我第一次接觸 CI

PichuChen commented 3 years ago

@PichuChen 不好意思，我對 Golang 的 naming 不太熟悉，希望這次都更正到了！ 想請問一下是需要您那邊幫忙跑 CI 測試還是 commit 一段時間就會自動檢測呢？ 有點好奇，我第一次接觸 CI

目前是我手動按的，好像是GitHub政策有變，所以沒有送過PR的人他不會自動跑CI

y2468101216 commented 3 years ago

我看不太懂 Test_getUserFlagAllByID 不過其他的沒問題

LGTM

y2468101216 commented 3 years ago

@PichuChen 不好意思，我對 Golang 的 naming 不太熟悉，希望這次都更正到了！ 想請問一下是需要您那邊幫忙跑 CI 測試還是 commit 一段時間就會自動檢測呢？ 有點好奇，我第一次接觸 CI

目前是我手動按的，好像是GitHub政策有變，所以沒有送過PR的人他不會自動跑CI

因為有發現有些不肖人士會利用 ci 挖礦