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

Add npm test #1368

Merged
merged 1 commit into from Jul 25, 2015
Merged

Add npm test #1368

merged 1 commit into from Jul 25, 2015

Conversation

jasnell
Copy link
Contributor

@jasnell jasnell commented Jul 25, 2015

Adds the npm test script to package.json.
The tests themselves do not need to be published to npm.

(This is all that's needed to get lodash working well with the http://github.com/nodejs/citgm tool)

Adds the npm test script to package.json.
The tests themselves do not need to be published to npm.
jdalton added a commit that referenced this pull request Jul 25, 2015
@jdalton jdalton merged commit 1d57383 into lodash:master Jul 25, 2015
@jdalton
Copy link
Member

jdalton commented Jul 25, 2015

Thanks!

@jasnell
Copy link
Contributor Author

jasnell commented Jul 25, 2015

Appreciate it! Just confirmed that lodash in github master is working with the citgm tool perfectly with this.

@jdalton
Copy link
Member

jdalton commented Jul 26, 2015

This isn't exactly what the npm package would be as that's on the npm branch.

@jasnell
Copy link
Contributor Author

jasnell commented Jul 26, 2015

Yep, I'm going to be teaching the test tool about git branches soon. For now, this get's me over the initial hump and at least allows the lodash tests to be run, even if it's not exactly what we get from npm.

@lock
Copy link

lock bot commented Dec 27, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

None yet

2 participants