tool4ever
tool4ever
@Hanmac I have it almost fixed like this:  But I think there is another logic error in the GainTextOf$ logic: currently it would add the RE twice - once...
Looks like it's TestNG causing problems? Would they even offer a version that can support 4 JDK simultaneously... 🤔 Otherwise I think seeing all PR fail (partly) is too noisy...
Yea there should be no reason to keep it: Because of `Defined$ Self` etc. effects need their own checks anyway For the most part only changezone triggers should be able...
A simple example where things get messed up currently: - _Shelob, Child of Ungoliant_ trigger on the stack - return the dead creature to the battlefield - no token 💩...
Needs #117
until `noTimestampCheck` contains all API bugs like the one above can still happen
@Hanmac adding ``alternatives.addAll(getMayPlaySpellOptions(iSa, source, activator, altCostOnly));`` here https://github.com/Card-Forge/forge/blob/600da3e80bbba868881e25b45affe776a0a00475/forge-game/src/main/java/forge/game/GameActionUtil.java#L247 will now let you Blitz your commander but it doesn't work for MDFC backside yet
I don't understand these reports. This card (like many others) is clearly marked as `AI:RemoveDeck:All` You should be warned when you force that on it in a deck
I'm not really outraged... If you don't appreciate me telling you you're wasting your time since us developers (well really anybody) can easily search for the affected cards then that's...
In theory everything can be coded but unique stuff might not be worth the effort (what is "easy" for the AI depends on a few factors). But I don't think...