search

dougmoscrop / serverless-plugin-include-dependencies

MIT License
184 stars 38 forks source link

Improve error message for: Could not find ${moduleName} #58

Closed aychtang closed 3 years ago

aychtang commented 3 years ago

I had a build error where I had this message:

Error: [serverless-plugin-include-dependencies]: Could not find critters

And had no real idea what it meant until looking into the source code for this package, @dougmoscrop shall we change it to something like:

Error: [serverless-plugin-include-dependencies]: Could not find npm package ${moduleName}

Happy to make the change + pr if needed.

pakorn-tp commented 3 years ago

@aychtang I support this issue. I spend my time about 3 hours for fix error message "Could not find request"

aychtang commented 3 years ago

@pakorn-tp @dougmoscrop: Definitely seems like this can cause confusion. Since someone else has also had this issue now I'll create a fix PR

dougmoscrop commented 3 years ago

Thanks that would be great!

aychtang commented 3 years ago

@dougmoscrop: I've added a PR @ https://github.com/dougmoscrop/serverless-plugin-include-dependencies/pull/70.