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

Added test for RT 131381 #273

Merged
merged 2 commits into from Jun 1, 2017
Merged

Conversation

danielcliffordmiller
Copy link
Contributor

Wrote a test for RT 131381 that currently fails.
https://rt.perl.org/Public/Bug/Display.html?id=131381

@MasterDuke17
Copy link
Contributor

Two quick things. You need to increment the plan count (https://github.com/danielcliffordmiller/roast/blob/d4aea7748a4e62e6ed218ab7f7e9bb87f561d005/S12-subset/subtypes.t#L7) by the number of added tests . Also, if the test fails, you need to fudge it with a todo on the like before the lives-ok (e.g., #?rakudo todo 'RT #131381').

@zoffixznet zoffixznet merged commit a19c632 into Raku:master Jun 1, 2017
@zoffixznet
Copy link
Contributor

Thanks. I think there must be no space in the fudge tho, so it'd be:

#?rakudo todo 'blah blah'

zoffixznet added a commit that referenced this pull request Jun 1, 2017
@zoffixznet
Copy link
Contributor

@danielcliffordmiller BTW, add yourself to CREDITS: https://github.com/rakudo/rakudo/blob/nom/CREDITS

Currently, you'll be listed in release announcement as danielcliffordmiller

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