Skip to content

Conversation

@hsheth2
Copy link
Contributor

@hsheth2 hsheth2 commented Sep 18, 2025

Follows a similar pattern to the other pgnInJson parameters.

Note that adding a parameter in the middle of the list is technically a breaking change, but I suspect most people are using kwargs so it won't matter. Let me know if you want me to move the new arg to the end of the arg list.

Checklist when adding a new endpoint
  • Added new endpoint to the README.md
  • Ensured that my endpoint name does not repeat the name of the client. Wrong: client.users.get_user(), Correct: client.users.get()
  • Typed the returned JSON using TypedDicts in berserk/types/, example
  • Written tests for GET endpoints not requiring authentification. Documentation, example
  • Added the endpoint and your name to CHANGELOG.md in the To be released section (to be created if necessary)

Follows a similar pattern to the other pgnInJson parameters.

Note that adding a parameter in the middle of the list is technically a breaking change, but I suspect most people are using kwargs so it won't matter. Let me know if you want me to move the new arg to the end of the arg list.
@kraktus kraktus merged commit 52f4119 into lichess-org:master Sep 20, 2025
30 checks passed
@kraktus
Copy link
Member

kraktus commented Sep 20, 2025

Thanks!

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.

2 participants