einar/maubot-osc
1
0
Fork 0
Code Issues Pull requests Releases 5 Activity

Add hack for non positional arguments support in maubot

Browse source 
There is no concept of non-positional arguments in maubot, so we just use
"all" in case we want to skip something.
This commit is contained in:
Luca Beltrame 2022-01-23 11:36:46 +01:00
parent 32876558ae
commit ab006d8c2e
Signed by: einar
GPG key ID: 4707F46E9EC72DEC
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
oscbot/__init__.py
View file

@ -14,8 +14,6 @@ from maubot.handlers import command
from mautrix.util.config import BaseProxyConfig, ConfigUpdateHelper from mautrix.util.config import BaseProxyConfig, ConfigUpdateHelper
API_URL = "https://api.opensuse.org"
HEADER_TEMPLATE = """ HEADER_TEMPLATE = """
### Package status ### Package status
@ -78,7 +76,8 @@ class OSCBot(Plugin):
username = self.config["username"] username = self.config["username"]
password = self.config["password"] password = self.config["password"]
api_call = f"{API_URL}/build/{project}/_result" api_url = self.config["api_url"]
api_call = f"{api_url}/build/{project}/_result"
auth = aiohttp.BasicAuth(username, password) auth = aiohttp.BasicAuth(username, password)
@ -153,10 +152,16 @@ class OSCBot(Plugin):
repository: Optional[str] = None, repository: Optional[str] = None,
arch: Optional[str] = None) -> None: arch: Optional[str] = None) -> None:
# There is no concept of non-positional arguments in maubot
# So we just use "all" in case we want to skip something
if state == "all": if state == "all":
state = None state = None
if package == "all": if package == "all":
package = None package = None
if repository == "all":
repository = None
if arch == "all":
arch = None
response = await self.parse_status(project, package, state=state, response = await self.parse_status(project, package, state=state,
repo=repository, arch=arch) repo=repository, arch=arch)