marvk / chess

A chess library, chess engine, Lichess client, and UCI implementation written in Java
https://lichess.org/@/QueensGamBOT
24 stars 7 forks source link

Refactor: explicit the fact the account name should be lowercase #21

Open dehidehidehi opened 1 year ago

dehidehidehi commented 1 year ago

Great engine, thank you !

I would suggest making it explicit to provide a lowerCase account name when passing in the accountName into the LichessClientBuilder class.

Otherwise, the case sensitive equals() checks to determine who's turn it is (AI or player) always fails or returns null.

It was difficult for me to realize this.