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.

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

Add PyChess #1901

Merged
merged 4 commits into from
Jun 14, 2022
Merged

Add PyChess #1901

merged 4 commits into from
Jun 14, 2022

Conversation

Crilum
Copy link
Contributor

@Crilum Crilum commented May 27, 2022

For @Rak1ta's App Request - closes #1898

This installs the latest version from GitHub (1.0.3), as opposed to @Rak1ta's app which just installs the apt version (1.0.0-1.2).

This also has an update script.

@theofficialgman Are update scripts supposed to have a pi_apps_ver/_64/_32 variable, like the docs say?

@Crilum Crilum mentioned this pull request May 27, 2022
3 tasks
@cycool29
Copy link
Contributor

cycool29 commented May 27, 2022

@theofficialgman Are update scripts supposed to have a pi_apps_ver/_64/_32 variable, like the docs say?

No. The update_github_script.sh already does it.

@Crilum
Copy link
Contributor Author

Crilum commented May 28, 2022

@theofficialgman Are update scripts supposed to have a pi_apps_ver/_64/_32 variable, like the docs say?

No. The update_github_script.sh already does it.

Thanks.

@theofficialgman
Copy link
Collaborator

@theofficialgman Are update scripts supposed to have a pi_apps_ver/_64/_32 variable, like the docs say?

You can specify them but there is no need to and you really shouldn't as @cycool29 said, the script itself will automatically populate the variables as necessary.

Everything you have committed is correct.

@Botspot Botspot merged commit 635ab67 into Botspot:master Jun 14, 2022
@Botspot
Copy link
Owner

Botspot commented Jun 14, 2022

Just played my first match with Stockfish! (and lost)

@Crilum Crilum deleted the pychess-pr branch June 14, 2022 19:50
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.

PyChess
4 participants