search

program-book-club-for-fun / bookmark-client

0 stars 3 forks source link

Resolve #5: ESLint and Editorconfig settings #6

Closed peterhpchen closed 6 years ago

peterhpchen commented 6 years ago
peterhpchen commented 6 years ago

This PR surveys #5.

peterhpchen commented 6 years ago

補上prettier及stylelint

henry61024 commented 6 years ago

Scott 的 commit 怎麼也在裡面

7463761ee & 04af4f5d3

peterhpchen commented 6 years ago

是之前的吧，merge的時候都會有

henry61024 commented 6 years ago

但這兩個 commit 前後的 commit 都是跟 lint 有關的耶

peterhpchen commented 6 years ago

Pull的時候不是會拉已經commit的進來嗎?

henry61024 commented 6 years ago

但 bookmark client 的 dev 沒有這兩個 commit 耶

peterhpchen commented 6 years ago

應該是之前史考特commit後又有rebase造成的吧?

所以我應該要重來一次嗎?

henry61024 commented 6 years ago

看你覺得那兩個 commit 的 code 有沒有問題吧，沒問題的話就看你們囉

另外 local repository 應該可以用 issue 分支比較可以避免這個問題

像是用 5-eslint-and-setting 之類的

這樣之後 merge 後的那個 commit 訊息也會比較清楚

peterhpchen commented 6 years ago

應該要改掉的，再試試