Skip to content
This repository has been archived by the owner on Nov 29, 2017. It is now read-only.

Board and Scoring #11

Merged
merged 30 commits into from
Aug 20, 2017
Merged

Board and Scoring #11

merged 30 commits into from
Aug 20, 2017

Conversation

vapour101
Copy link
Owner

No description provided.

Tshiwela and others added 13 commits August 14, 2017 13:21
These files need to be shared by necessity so that everyones' code style is consistent
Added a new class "DrawCoords" to hold 2D coordinates for draw calls, this should increase the readability of the code.
Simple scoring now works and is tested
Added more tests and fixed an obscure bug that the tests revealed
Added a real game to the BoardScorer testsuite, confirming that the scorer works as intended.
Yet another test from a real game, however scoring this one is far more complicated.
@coveralls
Copy link

Coverage Status

Coverage increased (+6.5%) to 100.0% when pulling d152453 on board into 3769b38 on master.

@vapour101 vapour101 mentioned this pull request Aug 20, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+1.7%) to 95.161% when pulling 103b78a on board into 3769b38 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+1.7%) to 95.161% when pulling 9af57be on board into 3769b38 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+6.5%) to 100.0% when pulling 2843295 on board into 3769b38 on master.

Playing works fine, just need to implement the scoring state
@coveralls
Copy link

Coverage Status

Coverage increased (+6.5%) to 100.0% when pulling bc26d8b on board into 3769b38 on master.

@vapour101 vapour101 merged commit 91bfbac into master Aug 20, 2017
@vapour101 vapour101 deleted the board branch August 20, 2017 20:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants