Replies: 13 comments
-
|
em nghĩ là để lint sai lint thì nó hú để cho anh viết cho quen standards :)) |
Beta Was this translation helpful? Give feedback.
-
|
Hay là mình đi theo hướng 1 nhưng nó chỉ la làng lên cho anh chị em format, viết cho quen standards? :)) |
Beta Was this translation helpful? Give feedback.
-
Tối hôm qua agree on đi theo hướng đó, tức là làm 1 cái hook, nó sẽ lint, nhưng k fix. Cơ mà tối hôm qua em check lại thì có vẻ git pre-commit hook nó k push đc lên repo, phải set riêng từng người, k biết mọi ng chịu commit ko :( tạm thời em đang làm 1 cái là cài luôn cái lệnh cho mỗi lân start là nó tự test lint watch luôn. |
Beta Was this translation helpful? Give feedback.
-
|
Em thử xem qua husky chưa? |
Beta Was this translation helpful? Give feedback.
-
|
Em có check qua thử Husky, mà vấn đề là mình k có npm package ở folder gốc, mà chỉ có npm package ở 1 folder con. K lẽ phải chạy 2 lần |
Beta Was this translation helpful? Give feedback.
-
|
Well thì do có 2 projects trong 1 repo thôi. Em có thể xem thử https://typicode.github.io/husky/#/?id=custom-directory |
Beta Was this translation helpful? Give feedback.
-
Vậy thì có hơi weird ko, khi 1 folder con nó lại in control of cái gì xảy ra ngoài folder gốc... |
Beta Was this translation helpful? Give feedback.
-
|
1 repo 2 Projects con với 2 package.json files thì cũng weird rồi. |
Beta Was this translation helpful? Give feedback.
-
|
Yeah nên em k muốn nó weird hơn :-/ Nếu làm theo hướng anh nói có khi phải restructure lại thành lerna-style monorepo, r xong build docker theo hướng khác. Mà cái đó lại phải nghiên cứu cái Lerna nó ntn... and then nó rối như tơ nhện... |
Beta Was this translation helpful? Give feedback.
-
|
https://dev.to/krzysztofkaczy9/do-you-really-need-husky-247b hmm... |
Beta Was this translation helpful? Give feedback.
-
|
Anh nghĩ là em đang đi lòng vòng nếu muốn đi hướng 1. Vì kiểu gì cũng phải có 1 cách nào đó cài git-hook vào trước khi commit & push. 1 là 2-step installer hoặc 2 là cái sub-project enforce cho cái tổng bên ngoài. |
Beta Was this translation helpful? Give feedback.
-
|
Note: Đã add git-hook trong PR #20 |
Beta Was this translation helpful? Give feedback.
-
Tiếp nối cái discussion từ bên PR #23 . Có argument 2 hướng là ntn
Hướng 1:
Mục đích chính cho việc này là vì cứ push commit lên, xong cứ có 1 cái commit "Fix lint issue"
thì nhìn cái git log nó sẽ rất rối.
Hướng 2:
Không làm. Vì làm thì dev phải để cho ng ta hiểu tại sao lại phải lint. Vấn đề này nên là vấn đề từ trong tư tưởng,
chứ k phải nên rely vào tool.
Các anh em quyết định thế nào?
Beta Was this translation helpful? Give feedback.
All reactions