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.

Sign up for GitHub

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

Jump to bottom

Renamed hasMove to hasMark #6

Closed
wants to merge 1 commit into from
Closed

Renamed hasMove to hasMark #6

wants to merge 1 commit into from

Commits on Jan 30, 2015

  1. Renamed hasMove to hasMark 

    Since you are marking the squares and it returns true if the square was
marked.  To me if it has a Move, I don't know at first if it meant it
had a Move available or a Move already made.  Plus move feels more like
a chess term where you are moving pieces around.  Anyway, small rename
for readability. No hard feelings is the mark as verb, mark as noun
issue is worse naming wise for some folks.
    @rseymour
    rseymour committed Jan 30, 2015
    Configuration menu
    Copy the full SHA
    2c85d5f View commit details
    Browse the repository at this point in the history