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 "wrap" to exports. #489

Merged
merged 2 commits into from
Feb 20, 2016
Merged

Add "wrap" to exports. #489

merged 2 commits into from
Feb 20, 2016

Conversation

cahrens
Copy link
Contributor

@cahrens cahrens commented Feb 20, 2016

@stoeffel Here is my PR to fix the exports function.

@cahrens cahrens changed the title Add test for exports. Add "wrap" to exports. Feb 20, 2016
@@ -0,0 +1,9 @@
var _ = require('underscore');
var deepEqual = require('assert').deepEqual;
var s = require('../dist/underscore.string');
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you change this to

- var s = require('../dist/underscore.string');
+ var s = require('../');

otherwise this looks good.

@stoeffel
Copy link
Collaborator

btw. to run the tests locally you can just run npm test in your shell.

Christina Roberts added 2 commits February 20, 2016 09:33
@cahrens
Copy link
Contributor Author

cahrens commented Feb 20, 2016

@stoeffel I made the change you requested (also changed a usage of double quotes to single quotes, as it appears that you are standardizing on single quotes) and squashed the commit. Thanks for pushing out a release so quickly!

I took a look at #481 and #482, but don't really have the background to review them.

@stoeffel
Copy link
Collaborator

Thanks for the PR.

I took a look at #481 and #482, but don't really have the background to review them.

No problem. Thanks anyway.

stoeffel added a commit that referenced this pull request Feb 20, 2016
@stoeffel stoeffel merged commit 74f3e96 into esamattis:master Feb 20, 2016
@stoeffel
Copy link
Collaborator

finally published 3.3.4. had some troubles with the new build chain, but now it works fine.
// cc @epeli

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.

2 participants