Review fixes + fixed missing docstring

allegro · Aug 2, 2016 · ac49fd1 · ac49fd1
1 parent 8507c89
commit ac49fd1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/ralph/settings/base.py b/src/ralph/settings/base.py
@@ -7,6 +7,10 @@
 
 
 def os_env_true(var, default=''):
+    """Helper for converting env string into boolean.
+
+    Returns bool True for string values: '1' or 'true', or False otherwise.
+    """
     return os.environ.get(var, default).lower() in ('1', 'true')
 
 BASE_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))

diff --git a/src/ralph/settings/prod.py b/src/ralph/settings/prod.py
@@ -3,7 +3,7 @@
 
 from ralph.settings import *  # noqa
 
-DEBUG = os.environ.get('RALPH_DEBUG', False) 
+DEBUG = os_env_true('RALPH_DEBUG', False)
 
 STATICFILES_STORAGE = 'django.contrib.staticfiles.storage.ManifestStaticFilesStorage'  # noqa