heroku / heroku-buildpack-nodejs

Heroku's buildpack for Node.js applications.
https://devcenter.heroku.com/articles/buildpacks
MIT License
1.31k stars 2.63k forks source link

`pnpm` not used during `test` #1307

Closed joshwlewis closed 3 months ago

joshwlewis commented 3 months ago

Describe the bug

When using Heroku CI with a pnpm application, the test script attempts to run npm test. Ideally, this should run the pnpm equivalent command. It seems like there is a missing condition for pnpm here: https://github.com/heroku/heroku-buildpack-nodejs/blob/main/bin/test#L3-L8.