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

added more tests to t/ack-output.t #590

Merged
merged 1 commit into from Jan 3, 2016

Conversation

Projects
None yet
2 participants
@Varadinsky
Contributor

Varadinsky commented Jan 3, 2016

Per @petdance 's suggestion in #587 I decided to add some more tests.

Not sure if you still want to keep the old single hit tests though.

Let me know and I will remove them if needed.

@hoelzro

This comment has been minimized.

Show comment
Hide comment
@hoelzro

hoelzro Jan 3, 2016

Collaborator

Looks good to me! Thanks again @Varadinsky!

Collaborator

hoelzro commented Jan 3, 2016

Looks good to me! Thanks again @Varadinsky!

hoelzro added a commit that referenced this pull request Jan 3, 2016

@hoelzro hoelzro merged commit a43b196 into beyondgrep:dev Jan 3, 2016

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment