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

Fails but I have no idea why #717

Merged
merged 13 commits into from
Jul 4, 2016
Merged

Conversation

zethfoxster
Copy link
Member

this is getting frustrating.

trying to update fork to have changes from master
other={convoke} name(Convoke)
delve
other={delve}
they might be able to be added directly to the real manacost.

added an ability that grants an ability while the source remains tapped
grant ability grantend...

added dethrone
abilities=dethrone

added support of multitargeting to extra cost, it acts the same as normal multitargeting, repeats dopay() the effects for each.
… not allowed to select the source to pay for the cost.

as long as the cost are not similar, the source is a valid target.
{s}{s(creature|mybattlefield)}
or
{t}{t(creature)}
or when the card states specifically that the source is not valid such as 
"{t},sacrifice another creature you control:draw a card"
@xawotihs
Copy link
Member

xawotihs commented Jul 4, 2016

Don't worry with Travis, if it runs down to the 700+ tests, it means that at least linux, psp and android works.... it used to returns 0 fine but I probably broke it when I introduced the iosx+macosx build...

@zethfoxster zethfoxster merged commit 52988f1 into WagicProject:master Jul 4, 2016
@zethfoxster
Copy link
Member Author

I spent half of last night fixing, and all of today trying to find out why after all test passed it still told me it fails. can we work on a fix for it so it reports correctly. I was asked to present "passing" builds, and Ive done my part...but it saids failed in huge red letters at the top.

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.

2 participants