jenkins-bot submitted this change.

View Change

Approvals: Xqt: Looks good to me, approved jenkins-bot: Verified
[cleanup] Explicitly return None from ContextOption.result()

According to Xqt this method's pydoc is outdated and returning None is
the correct behavior [1]. This doesn't change anything - just minor
cleanup.

[1] https://gerrit.wikimedia.org/r/c/pywikibot/core/+/704637

Bug: T286403
Change-Id: Icc979089912bf5b75f2518cef03bc33a5d8b8d63
---
M pywikibot/bot_choice.py
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pywikibot/bot_choice.py b/pywikibot/bot_choice.py
index 89daa17..59aad26 100755
--- a/pywikibot/bot_choice.py
+++ b/pywikibot/bot_choice.py
@@ -242,9 +242,9 @@
self.end = end

def result(self, value: str) -> Any:
- """Add the delta to the context and output it."""
+ """Add the delta to the context."""
self.context += self.delta
- super().result(value)
+ return None

@property
def out(self) -> str:

To view, visit change 704637. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Change-Id: Icc979089912bf5b75f2518cef03bc33a5d8b8d63
Gerrit-Change-Number: 704637
Gerrit-PatchSet: 3
Gerrit-Owner: Damian <atagar1@gmail.com>
Gerrit-Reviewer: Xqt <info@gno.de>
Gerrit-Reviewer: jenkins-bot
Gerrit-MessageType: merged