Lint fixes.

* No need to define and return `__virtualname__` if not renaming the module. In this case, just return `True`.
* Don't shaddow the buil-in `set`
* Removed logging since it's not being used
This commit is contained in:
Pedro Algarvio 2014-02-18 01:16:50 +00:00
parent 9a59f962eb
commit 4560891971

View File

@ -6,25 +6,24 @@ of the current salt process.
# Import python libs
import os
import logging
# Import salt libs
from salt._compat import string_types
from salt.exceptions import SaltException
log = logging.getLogger(__name__)
__func_alias__ = {
'set_': 'set'
}
# Define the module's virtual name
__virtualname__ = 'environ'
def __virtual__():
'''
No dependency checks, so just return the __virtualname__
No dependency checks, and not renaming, just return True
'''
return __virtualname__
return True
def set(name, value):
def set_(name, value):
'''
Set the salt process environment variables.
@ -93,4 +92,3 @@ def set(name, value):
ret['changes'] = environ_ret
ret['comment'] = 'Environ values were set'
return ret