Skip to content
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

Almost there! #2

Open
wants to merge 94 commits into
base: master
Choose a base branch
from
Open

Almost there! #2

wants to merge 94 commits into from

Conversation

gellieb
Copy link

@gellieb gellieb commented Mar 8, 2015

Check at Board.rb

Our board looks pretty, the display works and will refresh after each turn. The pieces functionality is in tact. Each piece has the correct moves/path. We have a conversion method that converts the Letter/numbers from user input to corresponding array points.

We will sort out the classes into separate files so that it's more readable.

Things we still need to do

create a capture/remove method
check if white/black player is choosing their own piece
check if player is picking a valid move (piece that doesn't have zero moves, else try again)
check if game ends (are both kings still on the board?)

@queerviolet @zencephalon @steppinlo @mbouzi @ebutler90

Eveanandi Butler & Angel Baek & Malcolm Bouzi & Stephanie Lo and others added 30 commits March 5, 2015 11:46
@@ -0,0 +1,324 @@
WIDTH = 8

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the use of constants. At this point one might consider wrapping everything in a module.

@zencephalon
Copy link

@gellieb @steppinlo @mbouzi @ebutler90 ILUVU, nice work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants