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

Adds a script in the package.json file to make running tests a little #2

Merged
merged 2 commits into from Mar 7, 2013

Conversation

qubyte
Copy link
Contributor

@qubyte qubyte commented Mar 7, 2013

easier (global install of mocha may be avoided). Updates README
accordingly.

easier (global install of mocha may be avoided). Updates README
accordingly.

$ npm install -g mocha

To run the unit tests, use the following command. Ensure that your code has an
appropriate set of unit tests which all pass.
With globally installed mocha, running all tests is done with

This comment was marked as spam.

This comment was marked as spam.

@rakyll
Copy link
Contributor

rakyll commented Mar 7, 2013

Could you also please take a look at the Google's Contributor License Agreements? Before I merge your changes, you need to sign one of them:

  • If you are an individual writing original source code and you're sure you
    own the intellectual property, then you'll need to sign an
    individual CLA.
  • If you work for a company that wants to allow you to contribute your work
    to this client library, then you'll need to sign a
    corporate CLA.

@qubyte
Copy link
Contributor Author

qubyte commented Mar 7, 2013

I signed the electronic Individual CLA, so hopefully that will be processed soon. Thanks for making this project available, I'm looking forward to playing with it!

@ghost ghost assigned rakyll Mar 7, 2013
rakyll pushed a commit that referenced this pull request Mar 7, 2013
Adds a script in the package.json file to make running tests a little
@rakyll rakyll merged commit 8100c5b into googleapis:master Mar 7, 2013
@rakyll
Copy link
Contributor

rakyll commented Mar 7, 2013

It's processed immediately. Thanks for the PR! I was about to make a similar change to be able to setup a Travis build.

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.

None yet

2 participants