draft icon indicating copy to clipboard operation
draft copied to clipboard

Please don't merge this: Implement Winston draft

Open manpages opened this issue 9 years ago • 4 comments

Hey, this is a rather dirty implementation of a winston draft.

Lacks advanced configuration options (Pool API should be refactored for that); lacks meta information; lacks deck hashing; lacks end game condition (now pool just get exhausted and all the piles will show 0 forever).

This PR, however, gives a person with more free time than myself a good overview about adding a new game type to the platform and a starting pointer about how to implement winston draft.

This PR also accidentally fixes #146.

manpages avatar Mar 20 '16 22:03 manpages

cool. can you give a prose overview?

aeosynth avatar Mar 21 '16 08:03 aeosynth

cool. can you give a prose overview?

I will when I have time. Most likely next weekend. :(

manpages avatar Mar 21 '16 10:03 manpages

alright, i got around to looking at this, and it does highlight a need for refactoring. did you only show one pile for ease of implementation, or is looking at previous piles disallowed/discouraged?

edit: ah nvm, the previous pile gets an unknown card added, so ofc you shouldn't see it

i think this has to wait for my rewrite

aeosynth avatar Apr 19 '16 21:04 aeosynth

Cool, I don't have any plans for doing anything with this implementation for a good while, but eventually I would like to see that finished, either through my work, or by mentoring someone else.

Sorry that I didn't post an overview, I'm way too busy these days.

manpages avatar Apr 20 '16 16:04 manpages