Update docstring and fix ValueError on incorrect condition
This commit is contained in:
parent
d96ed7cfde
commit
bf2adbd49e
@ -39,7 +39,7 @@ def gen_cmdline(operation, db, opts, *pkgs):
|
|||||||
:rtype: list
|
:rtype: list
|
||||||
:raises: ValueError if operation was invalid
|
:raises: ValueError if operation was invalid
|
||||||
"""
|
"""
|
||||||
if not operation == 'add' or operation == 'remove':
|
if not operation == 'add' or not operation == 'remove':
|
||||||
raise ValueError('Invalid operation was requested')
|
raise ValueError('Invalid operation was requested')
|
||||||
|
|
||||||
cmd = [f"/usr/bin/repo-{operation}"]
|
cmd = [f"/usr/bin/repo-{operation}"]
|
||||||
@ -74,6 +74,7 @@ def repo_add(operation, db, *pkgs, opts=None):
|
|||||||
then pkgs is the name of the packages to remove from the database.
|
then pkgs is the name of the packages to remove from the database.
|
||||||
:type pkgs: any iterable
|
:type pkgs: any iterable
|
||||||
:raises: RepoAddError if repo-add failed
|
:raises: RepoAddError if repo-add failed
|
||||||
|
:raises: ValueError if an invalid operation was requested
|
||||||
"""
|
"""
|
||||||
if operation == 'add':
|
if operation == 'add':
|
||||||
syslog.info('Adding packages to database')
|
syslog.info('Adding packages to database')
|
||||||
|
Loading…
x
Reference in New Issue
Block a user