SkyeYoung commented on PR #2569: URL: https://github.com/apache/apisix-dashboard/pull/2569#issuecomment-1210074023
@FangSen9000 But you don't keep the file pnpm-lock.yaml, which leads to two problems for developers who want to use it. 1. there are multiple versions of the dependencies used, which can lead to inconsistent versions being used by developers, making the entire software environment unstable. 2. The need to exclude pnpm-lock.yaml for each commit makes it more difficult for developers who are just starting out to contribute. The above problems make it difficult to use "pnpm as an optional package manager", which leads to no one using pnpm as a package manager, and ultimately makes the whole PR ineffective. On reflection, it is also very difficult to maintain dependencies for both package managers, so I suggest you try replacing yarn with pnpm instead of trying to make them co-exist. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
