From 7a3fd83570992501ef9e0fc359fb9a60645fee09 Mon Sep 17 00:00:00 2001 From: Bibo-Joshi Date: Sat, 16 Jan 2021 13:57:25 +0100 Subject: [PATCH] Add PR Template (#2299) * Add PR template * Apply suggestions from code review Co-authored-by: Poolitzer <25934244+Poolitzer@users.noreply.github.com> * Some more * reformulate * Apply suggestions from code review Co-authored-by: Poolitzer <25934244+Poolitzer@users.noreply.github.com> Co-authored-by: Poolitzer <25934244+Poolitzer@users.noreply.github.com> --- .github/CONTRIBUTING.rst | 2 +- .github/pull_request_template.md | 29 +++++++++++++++++++++++++++++ 2 files changed, 30 insertions(+), 1 deletion(-) create mode 100644 .github/pull_request_template.md diff --git a/.github/CONTRIBUTING.rst b/.github/CONTRIBUTING.rst index bfd577714..a86878bec 100644 --- a/.github/CONTRIBUTING.rst +++ b/.github/CONTRIBUTING.rst @@ -91,7 +91,7 @@ Here's how to make a one-off code change. Once the process terminates, you can view the built documentation by opening ``docs/build/html/index.html`` with a browser. - - Add ``.. versionadded:: version``, ``.. versionchanged:: version`` or ``.. deprecated:: version`` to the associated documentation of your changes, depending on what kind of change you made. This only applies if the change you made is visible to an end user. + - Add ``.. versionadded:: version``, ``.. versionchanged:: version`` or ``.. deprecated:: version`` to the associated documentation of your changes, depending on what kind of change you made. This only applies if the change you made is visible to an end user. The directives should be added to class/method descriptions if their general behaviour changed and to the description of all arguments & attributes that changed. - For consistency, please conform to `Google Python Style Guide`_ and `Google Python Style Docstrings`_. diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md new file mode 100644 index 000000000..da032a0ee --- /dev/null +++ b/.github/pull_request_template.md @@ -0,0 +1,29 @@ + + +### Checklist for PRs + +- [ ] Added `.. versionadded:: version`, `.. versionchanged:: version` or `.. deprecated:: version` to the docstrings for user facing changes (for methods/class descriptions, arguments and attributes) +- [ ] Created new or adapted existing unit tests +- [ ] Added myself alphabetically to `AUTHORS.rst` (optional) + + +### If the PR contains API changes (otherwise, you can delete this passage) + +* New classes: + - [ ] Added `self._id_attrs` and corresponding documentation + - [ ] `__init__` accepts `**_kwargs` + +* Added new shortcuts: + - [ ] In `Chat` & `User` for all methods that accept `chat/user_id` + - [ ] In `Message` for all methods that accept `chat_id` and `message_id` + - [ ] For new `Message` shortcuts: Added `quote` argument if methods accepts `reply_to_message_id` + - [ ] In `CallbackQuery` for all methods that accept either `chat_id` and `message_id` or `inline_message_id` + +* If relevant: + + - [ ] Added new constants at `telegram.constants` and shortcuts to them as class variables + - [ ] Added new handlers for new update types + - [ ] Added new filters for new message (sub)types + - [ ] Added or updated documentation for the changed class(es) and/or method(s)