{"comment":{"url":"https://api.github.com/repos/scrapinghub/frontera/comments/11529132","html_url":"https://github.com/scrapinghub/frontera/commit/337100cb0ae9ade67e79693af7f9f863244df722#commitcomment-11529132","id":11529132,"user":{"login":"sibiryakov","id":9044124,"avatar_url":"https://avatars.githubusercontent.com/u/9044124?v=3","gravatar_id":"","url":"https://api.github.com/users/sibiryakov","html_url":"https://github.com/sibiryakov","followers_url":"https://api.github.com/users/sibiryakov/followers","following_url":"https://api.github.com/users/sibiryakov/following{/other_user}","gists_url":"https://api.github.com/users/sibiryakov/gists{/gist_id}","starred_url":"https://api.github.com/users/sibiryakov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sibiryakov/subscriptions","organizations_url":"https://api.github.com/users/sibiryakov/orgs","repos_url":"https://api.github.com/users/sibiryakov/repos","events_url":"https://api.github.com/users/sibiryakov/events{/privacy}","received_events_url":"https://api.github.com/users/sibiryakov/received_events","type":"User","site_admin":false},"position":4,"line":3,"path":"frontera/tests/test_settings.py","commit_id":"337100cb0ae9ade67e79693af7f9f863244df722","created_at":"2015-06-04T20:08:51Z","updated_at":"2015-06-04T20:08:51Z","body":"This line, indicates the problem pretty much. IMO, it's ambiguous to have two settings classes with such names. The purpose of them isn't clear from their names or paths. Maybe we don't need ``Settings`` class anymore? Or maybe it makes more sense to rename your current ``FronteraSettings`` to ``Settings``, IMO it looks clearer."}}