Conversation
xaved88
left a comment
There was a problem hiding this comment.
Thanks for the PR! So the reason that we didn't include True Grit a while back was because of the randomness in the un-upgraded version. Unless I'm mistaken (which can totally be the case, I've not done much with this project in over a year), random handling with cards like that wasn't ever accounted for and so this could lead to the bot being very optimistic in its use of the un-upgraded version.
It's probably better to have it in there, but maybe we should add in a comment to note this expected unexpected behaviour?
|
Yes, you're completely correct. I ran into the issue you mention when testing (on a new strategy) after creating the PR. I did later code some logic to handle random exhaust. I will see if I can combine some logic in one PR / add the comment as suggested. |
This just makes the card playable via combat, but not added to any strategy