-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release cycle #24
Comments
I guess I didn't explicitly say above - I think we are basically there for first release. Let's start getting others to help us out!!! |
Let me wrangle the non-lookup card evaluator I have (it does both 5 and 7 card evaluations using [Card]) into a PR and let's see how it looks. Taking a look at readme and properties ... |
Thank you for the great review and feedback @tonyday567 I think we should give ourselves a timeline for the release. I think it is not unreasonable to suggest for ourselves that we release on Thursday/Friday of this week (in 4/5 days). What do you think? |
I pushed the module unhiding PR - I needed a baseline commit for poker-fold to catch up with where poker-base is at right now. We're done on API changes for the release? Ok, if there's more to come, but just trying to firm things up and concentrate on polishing. |
@tonyday567 I feel good to go. I've got full test coverage on Poker.Cards and Poker.Game. Technically Poker.Game has no tests yet, but only because the doc-tests are extensive and I'll add tests for Position once that design is actually finalised (I don't know whether or not it is yet). Some notable wrinkles:
|
I've also gone through a lot of the docs - but it would be good to do one revision of docs prior to release - likely tomorrow... |
ok! Will respond to each issue about in their threads, but I vote for freezing everything prior to release, except for docs, haddocks and bugs (which I don't know of any). Will concentrate on getting CI right. |
Are you planning a post on release? |
Okay - library frozen! I don't know - I guess it would be good for us to include a bunch of examples from other libraries, say Let me know what you think about a post. I'm not sure whether/what kind of introduction |
One place that might be good for examples would be monetary quantities... |
I'm happy for a low-key start, and was actually thinking a slow buildup is a lot better than starting with something splashy. If we did go with a post, something like "Why haskell and poker coding are a good mix" might generate some coder interest. The monetary story is a good example of why, I could add in the mmap library thing (I used a library that hadnt been updated in 10 years and it worked perfectly etc), pattern synonyms giving us zero-cost abstraction and flexibility in future design. |
I like your comment - let's go with something like:
Something mildly related - it might be good to get an open source poker website up and running. |
Co-opting this thread to discuss the aftermath of release (such as issues as they arise and possible blog post etc.) |
We already have a milestone for this, but I thought I'd find a spot to have text to discuss when we should release.
I just uploaded a couple of PRs that I think solve some of the current issues with the package:
The remaining issue I see is 7-card evaluation. I think it would be fine to elide this addition for the first release? What do you think? We can just link to poker-fold as a WIP project that allows people to have something to look at. I believe in particular you said that you have a quite-accurate (though not tested) inefficient hand calculation method right?
The text was updated successfully, but these errors were encountered: