Skip to content

Conversation

@juanfgarcia
Copy link

No description provided.

@coveralls
Copy link

Coverage Status

Coverage decreased (-2.0%) to 64.557% when pulling 8705c17 on juanfgarcia:master into 924578e on freeeve:master.

Copy link
Owner

@freeeve freeeve left a comment

Choose a reason for hiding this comment

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

Thanks!

@freeeve freeeve merged commit 138f8c2 into freeeve:master Feb 9, 2020
}

// makeMove takes a move in UCI format (e.g e2e4 e5e7...) and tells the engine to set the position
func (eng *Engine) makeMove(move string) error{
Copy link
Owner

Choose a reason for hiding this comment

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

this should probably be a public method.

Copy link
Owner

Choose a reason for hiding this comment

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

aac5d09 I renamed it--thanks for the contribution!

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.

3 participants