phodal / adr

Architecture Decision Records in Node.js with Reporter, supported Windows, GNU/Linux, macOS - 轻量级架构决策记录工具
https://phodal.github.io/adr/
MIT License
263 stars 52 forks source link

removed package-lock.json to avoid dep conflict #37

Closed AmirAsghary closed 2 years ago

AmirAsghary commented 2 years ago

Including both package-lock.json and yarn.lock is known to cause dependency consistency issues. I'm in favor of using yarn over npm. I removed the package-lock.json file in this PR.

Ref: https://classic.yarnpkg.com/blog/2018/06/04/yarn-import-package-lock/

AmirAsghary commented 2 years ago

Yarn tends to be more performant and causes fewer headaches overall, that's why I prefer yarn.

Npm, when compared with yarn, has more users. Maybe keeping npm and removing yarn make the repo more accessible to contributors.

phodal commented 2 years ago

thanks