search

Dirk94 / ChessAI

Chess AI written in Python
97 stars 27 forks source link

Add castling #9

Closed phi12ip closed 1 year ago

phi12ip commented 2 years ago

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 commented 2 years ago

Can this be merged?

Dirk94 commented 1 year ago

@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 🚀

phi12ip commented 1 year ago

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

Cheers, Phil