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.

Sign up for GitHub

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

Jump to bottom

add a test documenting #1801 #1802

Merged
merged 1 commit into from
Jul 24, 2015
Merged

add a test documenting #1801 #1802

merged 1 commit into from
Jul 24, 2015

Conversation

tbg
Copy link
Member

@tbg tbg commented Jul 24, 2015

it would currently fail, so it is skipped.

@tbg
add a test documenting cockroachdb#1801
1c3aa5a 
it would currently fail, so it is skipped.
@tbg tbg added the PTAL label Jul 24, 2015
@tamird
Copy link
Contributor

tamird commented Jul 24, 2015

LGTM. I would really like it if testing had a facility for annotating a test as an expected failure (failing the test if it happens to pass).

tbg added a commit that referenced this pull request Jul 24, 2015
@tbg
Merge pull request #1802 from tschottdorf/yacc_panic
8b4e232 
add a test documenting #1801
@tbg tbg merged commit 8b4e232 into cockroachdb:master Jul 24, 2015
@tbg tbg removed the PTAL label Jul 24, 2015
@tbg tbg deleted the yacc_panic branch July 24, 2015 17:23
@tbg tbg mentioned this pull request Jul 24, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tbg @tamird