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

Allow setStopRow() to work with an empty test table. #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Allow setStopRow() to work with an empty test table. #16

wants to merge 1 commit into from

Conversation

axiak
Copy link

@axiak axiak commented Sep 11, 2013

@kryzthov
Copy link

Hey Michael,
Sorry for not noticing this earlier, we've moved code reviews to https://review.kiji.org
Thanks for this fix, I'll merge it soon.
C.

@axiak
Copy link
Author

axiak commented Sep 17, 2013

I tried using that site and no matter what I did it rejected my patch :(

@hougs
Copy link

hougs commented Sep 18, 2013

In what way did it "reject your patch"? If you get an error that says file <path/to/file> not found in repo, often you can use the --full-index flag when generating the diff to fix the problem.

@axiak
Copy link
Author

axiak commented Sep 18, 2013

let me try the --full-index flag and I'll let you know.

@axiak
Copy link
Author

axiak commented Sep 18, 2013

Full-index solved the problem: https://review.kiji.org/r/1126/

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.

3 participants