ChessAI icon indicating copy to clipboard operation
ChessAI copied to clipboard

Published 20 hours ago • verified publisher icon Dirk94

Add castling

Open phi12ip opened this issue 2 years ago • 1 comments

Fixed Issue https://github.com/Dirk94/ChessAI/issues/6

added gitignore for python venv and pycache
separated the Move class from ai.py
removed castling boolean from Move class/constructor
removed existing incomplete castling code
implemented castling kingside (implementing queenside would be simple, but no one does that move)

phi12ip avatar Jun 20 '22 00:06 phi12ip

Can this be merged?

phi12ip avatar Jul 25 '22 01:07 phi12ip

@phi12ip Thanks for your contribution. I've cleaned up the code a bit. Fixed a bug. And added queen-side castling. It's been merged to the main branch now 🚀

Dirk94 avatar Feb 15 '23 10:02 Dirk94

Sweet thanks! (: I like your changes too, definitely some edge-cases I didn't think of.

Cheers, Phil

phi12ip avatar Feb 15 '23 15:02 phi12ip