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 more tests #6

Merged
merged 1 commit into from
Oct 13, 2015
Merged

add more tests #6

merged 1 commit into from
Oct 13, 2015

Conversation

bcoe
Copy link
Contributor

@bcoe bcoe commented Oct 13, 2015

I've added some more tests, increasing the test coverage significantly. In the process I've nuked some dead code.

@sindresorhus could you perhaps add me as an owner to the wrap-ansi npm module, and I'll work on getting this module pulled into cliui -> yargs -> finally gulp.

reviewers: @dthree, @sindresorhus

@sindresorhus
Copy link
Member

could you perhaps add me as an owner to the wrap-ansi npm module

Done. Use np for publishing ;)

bcoe added a commit that referenced this pull request Oct 13, 2015
@bcoe bcoe merged commit 114f4d8 into master Oct 13, 2015
@bcoe bcoe deleted the moar-tests branch October 13, 2015 04:59
@dthree
Copy link
Contributor

dthree commented Oct 13, 2015

👍 lgtm

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

3 participants