Add the possibility to keep the cards of some players

[LucasColas]

No description provided.

[LucasColas]

I don't know how to how to fix that :
texasholdem/game/game.py:1279:4: R0912: Too many branches (16/12) (too-many-branches)
I tried to fix that with several solutions. But it's still here.

[LucasColas]

Another thing I don't understand is about test. I have tests that fail related to game._deck. Sometimes _deck is NoneType. But I don't understand why.

[SirRender00]

@LucasColas I believe the error is from before the prehand where there is no deck attribute

[LucasColas]

Yes,
We can use deck I think.

[codecov]

Codecov Report

Merging #203 (1390396) into main (f4b8dda) will decrease coverage by 0.08%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main     #203      +/-   ##
==========================================
- Coverage   90.97%   90.89%   -0.08%     
==========================================
  Files          18       18              
  Lines        1174     1175       +1     
==========================================
  Hits         1068     1068              
- Misses        106      107       +1     

Files Changed	Coverage Δ
texasholdem/game/game.py	92.97% <66.66%> (-0.20%)	⬇️

[LucasColas]

I don't understand. Whever I run poetry run black . all files are left unchanged.

[LucasColas]

Everything is good for pytest, pylint and black.

[SirRender00]

Everything is good for pytest, pylint and black.

Nice. I could see about adding a test when I get back from vacation next week. Feel free as well to take a crack at adding a new test case. Perhaps in the tests.game.conftest copy predicate (which I have right now running random games and each predicate is run before and after each hand state).

[SirRender00]

I think I would prefer to have the default be that all the player cards are the same. Here, it's that all the cards are different.

[SirRender00]

@LucasColas Did your latest change do anything for this?

[LucasColas]

ok, right.