Updates to Spades

[i-Madsen]

Now can get/set scores and set a bonus reward for winning/losing overall game to returns.

[lanctot]

Hi @i-Madsen, thanks for the updates!

Can you pull changes from master and merge them into your branch? I had to make some cosmetic changes to your code to adhere to the required style. (If you can keep these persistent and apply them elsewhere too, that'd be great.)

Once any conflicts are resolved, please push the merge commit.

[lanctot]

Also, note: I will soon not have access to the Internet for the next 8-10 days.. so apologies for any delays in responses.

[i-Madsen]

Ok, think I've got it all merged correctly and pushed once you get back. Let me know if I missed anything.

Edit: I do see it's failing checks on here, but I'm not what's different that's causing it.

[lanctot]

Hi @i-Madsen,

Ok the CI is fixed now as of #1250

Can you pull from master and push the merge commit?

[i-Madsen]

@lanctot Ok, should be merged now.

[lanctot]

@lanctot Ok, should be merged now.

Ok seems there are some test failures for legitimate reasons (a function is marked void but I guess it's returning a bool?) Can you take a look and push a fix?

[i-Madsen]

@lanctot Looks like it's all passing now (not sure how I missed that before).