Usability fixes in readme and package.json #436

Merged
merged 1 commit into from Mar 31, 2013

Conversation

Projects
None yet
2 participants
@1602
Contributor

1602 commented Mar 31, 2013

  • shorter npm install for git
  • git url in package.json: it useful when user come from npmjs.org/packages/node-mysql. Currently it says "Repository (undefined)" but it should link to github.
Usability of readme in package.json
- shorter npm install for git
- git url in package.json
@@ -15,7 +15,7 @@ Sometimes I may also ask you to install the latest version from Github to check
if a bugfix is working. In this case, please do:
```
-npm install git://github.com/felixge/node-mysql.git
+npm install felixge/node-mysql

This comment has been minimized.

Show comment Hide comment
@felixge

felixge Mar 31, 2013

Collaborator

So npm assumes git/github for this? ...

@felixge

felixge Mar 31, 2013

Collaborator

So npm assumes git/github for this? ...

This comment has been minimized.

Show comment Hide comment
@1602

1602 Mar 31, 2013

Contributor

Yep, and it makes life a little bit easier :)

@1602

1602 Mar 31, 2013

Contributor

Yep, and it makes life a little bit easier :)

felixge added a commit that referenced this pull request Mar 31, 2013

Merge pull request #436 from 1602/master
Usability fixes in readme and package.json

@felixge felixge merged commit 0aac74a into mysqljs:master Mar 31, 2013

1 check passed

default The Travis build passed
Details
@felixge

This comment has been minimized.

Show comment Hide comment
@felixge

felixge Mar 31, 2013

Collaborator

Thx for the patch!

Collaborator

felixge commented Mar 31, 2013

Thx for the patch!

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