Closed knutole closed 1 year ago
The coverage rate went from 96.75%
to 94.03%
:arrow_down:
The branch rate is 89%
.
97.95%
of new lines are covered.
Thanks @kieferro, appreciated! Please remember to enable the necessary Github Pages settings before merge. 👍
@knutole Looks good, thanks for adding this (and updating the descriptions for some of the functions). Github says the branch had an error being deployed, not sure why it says this though.
This is probably the Github Pages permissions that need to be set. See description of PR on top for how to set this.
@kieferro Seems Github Pages are not enabled.
@knutole Thanks, I merged first and have rerun the workflow. Everything should be fine now.
Adds API documentation to Github Pages
Pieces
,Captures
andStockfishException
to bottom of file for better readability of documentation.Github Pages
Run pdoc to test locally:
pdoc -d google stockfish/models.py
Requires enabling Github Pages on
master
fordocs/
path:Failures in deploying to Github Pages below is due to missing permission to deploy to GP with branch on
pull_request
(for testing). Tested on my own repo, and it works is permissions are set.