[PATCH] trivial: Resolve pep8 issues
Stephen Finucane
stephen at that.guru
Thu Nov 24 20:26:25 AEDT 2016
flake8 3.2.0 updates the pycodestyle dependency to 2.2.0, which in
turn resolves a bug with E305. Upgrading results in a small number of
additional pep8 issues. Resolve these now.
Signed-off-by: Stephen Finucane <stephen at that.guru>
---
patchwork/admin.py | 24 ++++++++++++++++++++++++
patchwork/bin/pwclient | 5 ++++-
patchwork/models.py | 2 ++
patchwork/templatetags/syntax.py | 1 +
patchwork/views/xmlrpc.py | 1 +
5 files changed, 32 insertions(+), 1 deletion(-)
diff --git a/patchwork/admin.py b/patchwork/admin.py
index e80415a..d43cc4d 100644
--- a/patchwork/admin.py
+++ b/patchwork/admin.py
@@ -47,6 +47,8 @@ class UserProfileInline(admin.StackedInline):
class UserAdmin(BaseUserAdmin):
inlines = (UserProfileInline, )
+
+
admin.site.unregister(User)
admin.site.register(User, UserAdmin)
@@ -61,6 +63,8 @@ class ProjectAdmin(admin.ModelAdmin):
inlines = [
DelegationRuleInline,
]
+
+
admin.site.register(Project, ProjectAdmin)
@@ -74,11 +78,15 @@ class PersonAdmin(admin.ModelAdmin):
has_account.boolean = True
has_account.admin_order_field = 'user'
has_account.short_description = 'Account'
+
+
admin.site.register(Person, PersonAdmin)
class StateAdmin(admin.ModelAdmin):
list_display = ('name', 'action_required')
+
+
admin.site.register(State, StateAdmin)
@@ -87,6 +95,8 @@ class SubmissionAdmin(admin.ModelAdmin):
list_filter = ('project', )
search_fields = ('name', 'submitter__name', 'submitter__email')
date_hierarchy = 'date'
+
+
admin.site.register(Submission, SubmissionAdmin)
admin.site.register(CoverLetter, SubmissionAdmin)
@@ -104,6 +114,8 @@ class PatchAdmin(admin.ModelAdmin):
is_pull_request.boolean = True
is_pull_request.admin_order_field = 'pull_url'
is_pull_request.short_description = 'Pull'
+
+
admin.site.register(Patch, PatchAdmin)
@@ -111,6 +123,8 @@ class CommentAdmin(admin.ModelAdmin):
list_display = ('submission', 'submitter', 'date')
search_fields = ('submission__name', 'submitter__name', 'submitter__email')
date_hierarchy = 'date'
+
+
admin.site.register(Comment, CommentAdmin)
@@ -130,6 +144,8 @@ class SeriesAdmin(admin.ModelAdmin):
def received_all(self, series):
return series.received_all
received_all.boolean = True
+
+
admin.site.register(Series, SeriesAdmin)
@@ -148,6 +164,8 @@ class SeriesInline(admin.StackedInline):
class SeriesReferenceAdmin(admin.ModelAdmin):
model = SeriesReference
+
+
admin.site.register(SeriesReference, SeriesReferenceAdmin)
@@ -157,6 +175,8 @@ class CheckAdmin(admin.ModelAdmin):
exclude = ('date', )
search_fields = ('patch__name', 'project__name')
date_hierarchy = 'date'
+
+
admin.site.register(Check, CheckAdmin)
@@ -164,9 +184,13 @@ class BundleAdmin(admin.ModelAdmin):
list_display = ('name', 'owner', 'project', 'public')
list_filter = ('public', 'project')
search_fields = ('name', 'owner')
+
+
admin.site.register(Bundle, BundleAdmin)
class TagAdmin(admin.ModelAdmin):
list_display = ('name',)
+
+
admin.site.register(Tag, TagAdmin)
diff --git a/patchwork/bin/pwclient b/patchwork/bin/pwclient
index ce18e54..48d7be9 100755
--- a/patchwork/bin/pwclient
+++ b/patchwork/bin/pwclient
@@ -377,6 +377,7 @@ def patch_id_from_hash(rpc, project, hash):
sys.exit(1)
return patch_id
+
auth_actions = ['check_create', 'update']
@@ -606,7 +607,8 @@ def main():
config.read([CONFIG_FILE])
if not config.has_section('options') and os.path.exists(CONFIG_FILE):
- sys.stderr.write('%s is in the old format. Migrating it...' % CONFIG_FILE)
+ sys.stderr.write('%s is in the old format. Migrating it...' %
+ CONFIG_FILE)
old_project = config.get('base', 'project')
@@ -793,5 +795,6 @@ def main():
action_parser.print_help()
sys.exit(1)
+
if __name__ == "__main__":
main()
diff --git a/patchwork/models.py b/patchwork/models.py
index a27dda6..15a2936 100644
--- a/patchwork/models.py
+++ b/patchwork/models.py
@@ -172,6 +172,7 @@ def _user_saved_callback(sender, created, instance, **kwargs):
profile = UserProfile(user=instance)
profile.save()
+
models.signals.post_save.connect(_user_saved_callback, sender=User)
@@ -907,4 +908,5 @@ def _patch_change_callback(sender, instance, **kwargs):
notification.last_modified = datetime.datetime.now()
notification.save()
+
models.signals.pre_save.connect(_patch_change_callback, sender=Patch)
diff --git a/patchwork/templatetags/syntax.py b/patchwork/templatetags/syntax.py
index 341a181..27ff8c3 100644
--- a/patchwork/templatetags/syntax.py
+++ b/patchwork/templatetags/syntax.py
@@ -33,6 +33,7 @@ def _compile(value):
regex, cls = value
return re.compile(regex, re.M | re.I), cls
+
_patch_span_res = [_compile(x) for x in [
(r'^(Index:?|diff|\-\-\-|\+\+\+|\*\*\*) .*$', 'p_header'),
(r'^\+.*$', 'p_add'),
diff --git a/patchwork/views/xmlrpc.py b/patchwork/views/xmlrpc.py
index 0c3f581..6ef957c 100644
--- a/patchwork/views/xmlrpc.py
+++ b/patchwork/views/xmlrpc.py
@@ -133,6 +133,7 @@ class PatchworkXMLRPCDispatcher(SimpleXMLRPCDispatcher,
return response
+
dispatcher = PatchworkXMLRPCDispatcher()
# XMLRPC view function
--
2.7.4
More information about the Patchwork
mailing list