Một vài góp ý - Githubissues

vincentpham13 commented 3 years ago

Sau khi xem qua thì mình có 1 vài góp ý hen:

Tận dụng absolute import của typescript trong tsconfig.json
App configuration nên viết riêng ra và chỉ import vào module thôi
Implement DI (Dependency Injection), sử dụng library như Inversify
boilerplate này based Express.js + dùng Typescript nhưng chưa thấy bạn dùng type definition của express (@types/express), và hầu như external libs bên ngoài nào bạn cũng phải dùng type của nó vì bạn đang viết bằng Typescript
Về phần Logger thì bạn có thể dùng Winston or Morgan thay vì tự viết mà dùng console.<> như vậy
Cuối cùng là Boilerplate này dùng những tech/stack nào thì bạn nên update vào Readme.md và có 1 list features, vd như: support ORM with Postgresql or MongoDB. để tránh việc người ta clone về xong ko dùng được nhé.

cám ơn chủ thớt đã xem

KenLaam commented 3 years ago

@vincentpham13 chắc do bạn ấy post 2 links 1 lúc nên bạn vào nhầm, repo bên này mới là của typesript nè bạn: https://github.com/dangthuyenvuong/nodejs-typescript-project-template còn repo này là js thuần 😆

dangthuyenvuong commented 3 years ago

@vincentpham13 cám ơn bác nha, những góp ý rất cần thiết luôn

Về absolute import thì cả 2 mình đều có config để absolute import (typescript thì có sẵn) sử dụng NODE_PATH .env file để config
App configuration: Mình sẽ xem lại, mình đang làm hơi thủ công
Implement DI: Mình sẽ thêm trong lần nâng cấp kế tiếp
Về typescript là đúng ý bác dưới, mình tạo 2 project 1 javascript và 1 typescript cho những ai cần gì thì dùng đó https://github.com/dangthuyenvuong/nodejs-typescript-project-template, Mình cũng đang tìm hiểu cách tách riêng ra giữa javascript thuần và typescript để mõi khi người dùng sử dụng javascript muốn chuyển sang typescript chỉ cần down phần declare của typescript thôi ko cần phải down toàn bộ source code lại giống như express và @types/express
Logger thì để mình xem lại, mục đích của logger mình tự viết là vì giữa prod và test mình chuyển đổi giữa ghi log ra console hay là ghi log ra file
tech/stack: Cám ơn bạn, mình sẽ tập ghi rõ hơn vào README

dangthuyenvuong / nodejs-project-template