-
Notifications
You must be signed in to change notification settings - Fork 35
/bin/sh: 1: npm: not found #283
Comments
I have hit the same issue! I have mitigated by rolling back to the previous version
to
One thing to note to people doing this. If you are using the default action linked on the README, then the action will still be using the old one on the PR you create with this change. I had to bypass PR policies and merge this, it then fixed any open PR's I had. It would be great if the breaking change @alfieyfc has highlighted could be reverted ASAP as this is highly disruptive to anyone pinning to |
Hey there! Sorry for the inconvenience. Can you please try again, branch v3 is updated with the reverted version. @iamludal In the current implementation, users can specify a package name for their conventionalcommits preset. This package will be installed with NPM if not present. The way this is done is not supported by your implementation at #267. Let's work a bit more on this before releasing in a new major. |
Hi! Sorry for the inconvenience. @aslafy-z Let's release a new major when the change will be stable as you suggested. I've seen you reverted the commit, does the error still happens? Or this issue can now be closed? |
The issue should be gone now, can you confirm @JackRhodes @alfieyfc? Thank you |
Hi, I think you have deleted a release. I think it would be better if a new patch was released instead of removing the version. The new version was working on my repos and dependabot auto-merged the changes and now my workflows are broken because the release has been retracted. |
@thiagownt Noted. I re-created tags with #267 reverted as |
@aslafy-z thanks for that. It is all working now 💯 |
Hello, thanks for this great action!
We've been using this for quite some time without any issues. Suddenly, since recent hours, we started to see this error:

We suspect this may have to do with the latest release from #267. Was there anything intended for us to change on our side for using the latest release?
The text was updated successfully, but these errors were encountered: