From eb71173be3c320aa42b90c8246bb8678373cf557 Mon Sep 17 00:00:00 2001 From: Tim Abbott Date: Mon, 20 Jun 2016 08:19:54 -0700 Subject: [PATCH] lint: Fix warnings under confirmation/. --- confirmation/models.py | 2 +- confirmation/util.py | 2 +- confirmation/views.py | 2 +- tools/lint-all | 1 - 4 files changed, 3 insertions(+), 4 deletions(-) diff --git a/confirmation/models.py b/confirmation/models.py index 33a7d49f88..9c291f7aae 100644 --- a/confirmation/models.py +++ b/confirmation/models.py @@ -110,4 +110,4 @@ class Confirmation(models.Model): verbose_name_plural = _('confirmation emails') def __unicode__(self): - return _('confirmation email for %s') % self.content_object + return _('confirmation email for %s') % (self.content_object,) diff --git a/confirmation/util.py b/confirmation/util.py index 7d544b98fd..e4269157ca 100644 --- a/confirmation/util.py +++ b/confirmation/util.py @@ -9,4 +9,4 @@ from django.conf import settings def get_status_field(app_label, model_name): model = '%s.%s' % (app_label, model_name) mapping = getattr(settings, 'STATUS_FIELDS', {}) - return mapping.get(model, 'status') \ No newline at end of file + return mapping.get(model, 'status') diff --git a/confirmation/views.py b/confirmation/views.py index ff558dd490..222200668a 100644 --- a/confirmation/views.py +++ b/confirmation/views.py @@ -41,5 +41,5 @@ def confirm(request, confirmation_key): ] if obj: # if we have an object, we can use specific template - templates.insert(0, 'confirmation/confirm_%s.html' % obj._meta.model_name) + templates.insert(0, 'confirmation/confirm_%s.html' % (obj._meta.model_name,)) return render_to_response(templates, ctx, request=request) diff --git a/tools/lint-all b/tools/lint-all index 5c1afc19b7..fc57045a35 100755 --- a/tools/lint-all +++ b/tools/lint-all @@ -34,7 +34,6 @@ os.chdir(os.path.join(os.path.dirname(__file__), '..')) exclude = """ static/third -confirmation frontend_tests/casperjs zerver/migrations node_modules