Skip to content

Update package.json dependencies type#99

Open
TaeKimJR wants to merge 1 commit intogarycourt:masterfrom
TaeKimJR:patch-1
Open

Update package.json dependencies type#99
TaeKimJR wants to merge 1 commit intogarycourt:masterfrom
TaeKimJR:patch-1

Conversation

@TaeKimJR
Copy link
Copy Markdown

@TaeKimJR TaeKimJR commented Feb 7, 2020

The dependencies field of the package.json was a type of array, but npm specifies it as an object type.

This came up as I was trying to use electron-builder and others saw a similar issue with other packages:
electron-userland/electron-builder#3050

@bribrah
Copy link
Copy Markdown

bribrah commented Jun 7, 2021

Can you pleeeaaasssseee merge this

@vcgupta
Copy link
Copy Markdown

vcgupta commented Jun 25, 2021

This needs to be merged as many times build fails because of this. Please merge

@lihaiJett
Copy link
Copy Markdown

please merge

@ssleptsov
Copy link
Copy Markdown

@lihaiJett @vcgupta @bribrah if you still need it, I have pushed a fixed version of it https://www.npmjs.com/package/jsv-fixed

@1AoB
Copy link
Copy Markdown

1AoB commented Aug 7, 2024

⨯ node_modules.Dependency.Dependencies: ReadMapCB: expect { or n, but found [, error found in #10 byte of ...|ncies" : [],
"main|..., bigger context ...|/garycourt/JSV.git"
}
],
"dependencies" : [],
"main" : "lib/jsv.js",
"keywords" : ["json"|...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants