Repository navigation
Commit b5f43c8
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -36,9 +36,7 @@ def _env_config() -> List[str]: | |||
| 36 | 36 | ||
| 37 | 37 | HELPER_PREFIX = "helper:" | |
| 38 | 38 | ||
| 39 | - HELPER_ATTRIBUTES = [ | ||
| 40 | - "job_token", "http_password", "private_token", "oauth_token" | ||
| 41 | - ] | ||
| 39 | + HELPER_ATTRIBUTES = ["job_token", "http_password", "private_token", "oauth_token"] | ||
| 42 | 40 | ||
| 43 | 41 | class ConfigError(Exception): | |
| 44 | 42 | pass | |
@@ -202,11 +200,11 @@ def __init__( | |||
| 202 | 200 | pass | |
| 203 | 201 | ||
| 204 | 202 | def _get_values_from_helper(self): | |
| 205 | - """Update attributes, which may get values from an external helper program | ||
| 206 | - """ | ||
| 203 | + """Update attributes, which may get values from an external helper program""" | ||
| 207 | 204 | for attr in HELPER_ATTRIBUTES: | |
| 208 | 205 | value = getattr(self, attr) | |
| 209 | - if isinstance(value, str) and value.lower().strip().startswith(HELPER_PREFIX): | ||
| 206 | + _value_lower = value.lower().strip() | ||
| 207 | + if isinstance(value, str) and _value_lower.startswith(HELPER_PREFIX): | ||
| 210 | 208 | helper = value[len(HELPER_PREFIX) :].strip() | |
| 211 | 209 | value = subprocess.check_output([helper]).decode("utf-8").strip() | |
| 212 | 210 | setattr(self, attr, value) | |
0 commit comments