- Notifications
You must be signed in to change notification settings - Fork3.1k
Commit83807db
fix: clarify the file name to be used (#1170)
As I was following the steps in this page -https://docs.npmjs.com/creating-node-js-modules, I noticed the followingerror when using the published package in the test-directory:`Error: Cannot find module'D:\Learning\npm-docs\use-hello\node_modules\@andrewnessindev\hello\index.js'.Please verify that the package.json has a valid "main" entry`It turned out that I missed providing the main entry even though this[page already mentions to provideit](https://github.com/npm/documentation/blob/2e10e4ed46f0f58adf4c165e8ef21599bc3b325f/content/packages-and-modules/contributing-packages-to-the-registry/creating-node-js-modules.mdx?plain=1#L20).However, it might be more helpful if the docs clarify which file isreferred to when[ it says, "In thefile"](https://github.com/npm/documentation/blob/2e10e4ed46f0f58adf4c165e8ef21599bc3b325f/content/packages-and-modules/contributing-packages-to-the-registry/creating-node-js-modules.mdx?plain=1#L28).This is my first commit! I followed the instruction it the`contributing.md` file for the commit message. I did not see arequirement to raise an issue first! Please do let me know if I have toraise an issue first or if I have missed anything else.---------Co-authored-by: Gar <wraithgar@github.com>1 parent2e10e4e commit83807db
File tree
1 file changed
+1
-1
lines changed- content/packages-and-modules/contributing-packages-to-the-registry
1 file changed
+1
-1
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
25 | 25 |
| |
26 | 26 |
| |
27 | 27 |
| |
28 |
| - | |
| 28 | + | |
29 | 29 |
| |
30 | 30 |
| |
31 | 31 |
| |
|
0 commit comments
Comments
(0)