Empty version specifiers in package.json #740

Closed
opened 2017-07-20 08:19:10 +00:00 by the-spyke · 5 comments
the-spyke commented 2017-07-20 08:19:10 +00:00 (Migrated from github.com)

Please replace empty version specifiers like "mocha":"" at least with asterisks "*" . Npm always checks for newer version on-line on npm install in empty case, even if the package is already in node_modules. Also it shows the package as invalid on npm list. All this is not an issue with asterisk.

Please do this in sub packages too. Thank you.

Please replace empty version specifiers like `"mocha":""` at least with asterisks `"*"` . Npm always checks for newer version on-line on `npm install` in empty case, even if the package is already in `node_modules`. Also it shows the package as invalid on `npm list`. All this is not an issue with asterisk. Please do this in sub packages too. Thank you.
SheetJSDev commented 2017-07-29 00:25:46 +00:00 (Migrated from github.com)

@the-spyke js-xlsx 0.10.9 has properly pinned dependencies and devDependencies. We updated the tree so it should work properly now. Please test and let us know if there are any issues

@the-spyke js-xlsx 0.10.9 has properly pinned dependencies and devDependencies. We updated the tree so it should work properly now. Please test and let us know if there are any issues
the-spyke commented 2017-07-30 15:00:21 +00:00 (Migrated from github.com)

@SheetJSDev Looks like frac@1.0.6 package still has empty version specifiers. Other packages are good. Thanks.

@SheetJSDev Looks like `frac@1.0.6` package still has empty version specifiers. Other packages are good. Thanks.
the-spyke commented 2017-08-01 08:52:31 +00:00 (Migrated from github.com)

@SheetJSDev All looks fine now. Thank you.

@SheetJSDev All looks fine now. Thank you.
SheetJSDev commented 2017-08-01 08:55:12 +00:00 (Migrated from github.com)

@the-spyke we bumped the minor version number because we also changed a build step for the dist files -- all of the require statements were stripped out, as they were confusing webpack when trying to bundle the minified files. The node and standard webpack usage should not have been changed.

@the-spyke we bumped the minor version number because we also changed a build step for the dist files -- all of the require statements were stripped out, as they were confusing webpack when trying to bundle the minified files. The node and standard webpack usage should not have been changed.
the-spyke commented 2017-08-02 06:58:09 +00:00 (Migrated from github.com)

Yeah, works good. Thank you.

Yeah, works good. Thank you.
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sheetjs/sheetjs#740
No description provided.