charleskawczynski commented 3 years ago

This PR:

switches to returning two structs (one with few allocations, and another with more), but still allows users to use the base evaluate5 method.
Updates the documentation
Improves names
Creates more layers to share core code

codecov[bot] commented 3 years ago

Codecov Report

Merging #23 (faf33e8) into main (e95334b) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #23   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines          138       149   +11     
=========================================
+ Hits           138       149   +11

Impacted Files	Coverage Δ
src/HandTypes.jl	`100.00% <100.00%> (ø)`
src/PokerHandEvaluator.jl	`100.00% <100.00%> (ø)`
src/evaluate5.jl	`100.00% <100.00%> (ø)`

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e95334b...faf33e8. Read the comment docs.

charleskawczynski commented 3 years ago

bors r+

bors[bot] commented 3 years ago

Build succeeded:

charleskawczynski / PokerHandEvaluator.jl

Interface revamp #23

Codecov Report