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.

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

fix: install package@tag -g should work #40

Merged
merged 1 commit into from
Mar 5, 2016
Merged

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Mar 5, 2016

ignore not range version check

ignore not range version check
@fengmk2 fengmk2 added the bug label Mar 5, 2016
@fengmk2 fengmk2 added this to the 1.x milestone Mar 5, 2016
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @dead-horse to be a potential reviewer

@fengmk2
Copy link
Member Author

fengmk2 commented Mar 5, 2016

I need to add a test case for this change

@codecov-io
Copy link

Current coverage is 93.74%

Merging #40 into master will decrease coverage by -1.13% as of bafbbf4

@@            master     #40   diff @@
======================================
  Files           16      16       
  Stmts          702     703     +1
  Branches       123     122     -1
  Methods          0       0       
======================================
- Hit            666     659     -7
  Partial          0       0       
- Missed          36      44     +8

Review entire Coverage Diff as of bafbbf4

Powered by Codecov. Updated on successful CI builds.

dead-horse added a commit that referenced this pull request Mar 5, 2016
fix: install package@tag -g should work
@dead-horse dead-horse merged commit 12503fc into master Mar 5, 2016
@dead-horse dead-horse deleted the fix-install-global branch March 5, 2016 05:43
@dead-horse
Copy link
Member

没看到.. 你加了直接提交吧。。。

@fengmk2
Copy link
Member Author

fengmk2 commented Mar 5, 2016

1.2.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants