Skip to content

Commit cab9805

Browse files
Resolve linting messages in readthedocs.settings.*
1 parent 85c69ce commit cab9805

File tree

3 files changed

+19
-16
lines changed

3 files changed

+19
-16
lines changed

readthedocs/settings/base.py

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@
4343
STATIC_ROOT = os.path.join(SITE_ROOT, 'media/static/')
4444
STATIC_URL = '/static/'
4545
STATICFILES_DIRS = [os.path.join(SITE_ROOT, 'readthedocs', 'static')]
46-
#STATICFILES_FINDERS = ()
46+
# STATICFILES_FINDERS = ()
4747

4848
CACHES = {
4949
'default': {
@@ -56,7 +56,7 @@
5656

5757
LOGIN_REDIRECT_URL = '/dashboard/'
5858
FORCE_WWW = False
59-
#APPEND_SLASH = False
59+
# APPEND_SLASH = False
6060

6161
# Docker
6262
DOCKER_ENABLE = False
@@ -87,7 +87,8 @@
8787
USE_I18N = True
8888
USE_L10N = True
8989
SITE_ID = 1
90-
SECRET_KEY = 'replace-this-please'
90+
91+
SECRET_KEY = 'replace-this-please' # noqa: ignore dodgy check
9192

9293
ACCOUNT_ACTIVATION_DAYS = 7
9394

@@ -113,7 +114,7 @@
113114
'readthedocs.core.middleware.SubdomainMiddleware',
114115
'readthedocs.core.middleware.SingleVersionMiddleware',
115116
'corsheaders.middleware.CorsMiddleware',
116-
#'django.contrib.flatpages.middleware.FlatpageFallbackMiddleware',
117+
# 'django.contrib.flatpages.middleware.FlatpageFallbackMiddleware',
117118
)
118119

119120
AUTHENTICATION_BACKENDS = (
@@ -130,7 +131,9 @@
130131
ACCOUNT_AUTHENTICATION_METHOD = "username_email"
131132
SOCIALACCOUNT_EMAIL_VERIFICATION = "none"
132133

133-
CORS_ORIGIN_REGEX_WHITELIST = ('^http://(.+)\.readthedocs\.org$', '^https://(.+)\.readthedocs\.org$')
134+
CORS_ORIGIN_REGEX_WHITELIST = (
135+
'^http://(.+)\.readthedocs\.org$',
136+
'^https://(.+)\.readthedocs\.org$')
134137
# So people can post to their accounts
135138
CORS_ALLOW_CREDENTIALS = True
136139
CORS_ALLOW_HEADERS = (
@@ -212,7 +215,7 @@
212215
'allauth.socialaccount',
213216
'allauth.socialaccount.providers.github',
214217
'allauth.socialaccount.providers.bitbucket',
215-
#'allauth.socialaccount.providers.twitter',
218+
# 'allauth.socialaccount.providers.twitter',
216219
]
217220

218221
REST_FRAMEWORK = {
@@ -429,10 +432,10 @@
429432
'propagate': False,
430433
},
431434
# Uncomment if you want to see Elasticsearch queries in the console.
432-
#'elasticsearch.trace': {
433-
# 'level': 'DEBUG',
434-
# 'handlers': ['console'],
435-
#},
435+
# 'elasticsearch.trace': {
436+
# 'level': 'DEBUG',
437+
# 'handlers': ['console'],
438+
# },
436439

437440
# Default handler for everything that we're doing. Hopefully this
438441
# doesn't double-print the Django things as well. Not 100% sure how

readthedocs/settings/onebox.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,14 +44,14 @@
4444
SESSION_COOKIE_DOMAIN = None
4545

4646
SLUMBER_USERNAME = 'test'
47-
SLUMBER_PASSWORD = 'test'
47+
SLUMBER_PASSWORD = 'test' # noqa
4848
SLUMBER_API_HOST = 'http://localhost:8000'
4949

5050
WEBSOCKET_HOST = 'localhost:8088'
5151

5252
DONT_HIT_DB = False
53-
#PRODUCTION_DOMAIN = 'readthedocs.org'
54-
#USE_SUBDOMAIN = True
53+
# PRODUCTION_DOMAIN = 'readthedocs.org'
54+
# USE_SUBDOMAIN = True
5555

5656

5757
if not os.environ.get('DJANGO_SETTINGS_SKIP_LOCAL', False):

readthedocs/settings/sqlite.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,17 +17,17 @@
1717

1818
BROKER_URL = 'redis://localhost:6379/0'
1919
CELERY_RESULT_BACKEND = 'redis://localhost:6379/0'
20-
#CELERY_ALWAYS_EAGER = False
20+
# CELERY_ALWAYS_EAGER = False
2121

2222

2323
SESSION_COOKIE_DOMAIN = None
2424
SESSION_COOKIE_HTTPONLY = False
2525
CACHE_BACKEND = 'dummy://'
2626

2727
SLUMBER_USERNAME = 'test'
28-
SLUMBER_PASSWORD = 'test'
28+
SLUMBER_PASSWORD = 'test' # noqa: ignore dodgy check
2929
SLUMBER_API_HOST = 'http://localhost:8000'
30-
#GROK_API_HOST = 'http://localhost:5555'
30+
# GROK_API_HOST = 'http://localhost:5555'
3131
PRODUCTION_DOMAIN = 'localhost:8000'
3232

3333
WEBSOCKET_HOST = 'localhost:8088'

0 commit comments

Comments
 (0)