Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

npminstall update certain package #181

Closed
afc163 opened this issue Jan 16, 2017 · 5 comments · Fixed by #182
Closed

npminstall update certain package #181

afc163 opened this issue Jan 16, 2017 · 5 comments · Fixed by #182
Assignees
@fengmk2
Labels
bug

Comments

@afc163
Copy link
Contributor

afc163 commented Jan 16, 2017 •
edited
Loading 

$ cd some-empty-folder
$ tnpm install antd exeq
$ tnpm update exeq
$ ls node_modules/antd
ls: node_modules/antd: No such file or directory     =>    antd is missing

ref: ant-design/antd-tools#29

$ tnpm -v
[email protected] (/usr/local/lib/node_modules/tnpm/bin/tnpm.js)
[email protected] (/usr/local/lib/node_modules/tnpm/node_modules/.3.10.9@npm/lib/npm.js)
[email protected] (/usr/local/bin/node)
[email protected] (/usr/local/lib/node_modules/tnpm/node_modules/.2.16.0@npminstall/lib/index.js)
prefix=/usr/local (load prefix from: $ npm config get prefix)
darwin x64 16.1.0 
registry=http://registry.npm.alibaba-inc.com
@afc163 afc163 added the bug label Jan 16, 2017
@fengmk2
Copy link
Member

fengmk2 commented Jan 16, 2017

呃，连 package.json 都没有么。。。

@fengmk2
Copy link
Member

fengmk2 commented Jan 16, 2017

可以加多一步逻辑，就是 tnpm update exeq 内部再执行一遍 install，前提是需要有 package.json

@afc163
Copy link
Contributor Author

afc163 commented Jan 17, 2017

有没有 package.json 都一样。

@fengmk2
Copy link
Member

fengmk2 commented Jan 17, 2017

那我按有 package.json 修复一下，没有的话，目前搞不定。

fengmk2 added a commit that referenced this issue Jan 17, 2017
@fengmk2
fix: update cli should ignore package names args
8b1ef18 
closes #181
@fengmk2
Copy link
Member

fengmk2 commented Jan 17, 2017

tnpm update 直接忽略所有参数，根据 package.json 重新安装

@fengmk2 fengmk2 mentioned this issue Jan 17, 2017
Merged
fengmk2 added a commit that referenced this issue Jan 18, 2017
@fengmk2
fix: update cli should ignore package names args (#182)
9d4b9d7 
closes #181

And add warning tips when package.json not exists
@snyk-bot snyk-bot mentioned this issue Sep 13, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fengmk2 fengmk2

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: update cli should ignore package names args cnpm/npminstall
2 participants
@fengmk2 @afc163