diff --git a/.pylintrc b/.pylintrc index ca39510..4eeeae9 100644 --- a/.pylintrc +++ b/.pylintrc @@ -6,7 +6,6 @@ disable=fixme, arguments-renamed, attribute-defined-outside-init, consider-using-generator, - consider-using-set-comprehension, duplicate-code, isinstance-second-argument-not-valid-type, keyword-arg-before-vararg, diff --git a/src/wuttaweb/forms/schema.py b/src/wuttaweb/forms/schema.py index fba366b..ca62d8a 100644 --- a/src/wuttaweb/forms/schema.py +++ b/src/wuttaweb/forms/schema.py @@ -557,7 +557,7 @@ class RoleRefs(WuttaSet): auth.get_role_authenticated(session), auth.get_role_anonymous(session), } - avoid = set([role.uuid for role in avoid]) + avoid = {role.uuid for role in avoid} # also avoid admin unless current user is root if not self.request.is_root: diff --git a/src/wuttaweb/views/users.py b/src/wuttaweb/views/users.py index 4691a1e..eb5c858 100644 --- a/src/wuttaweb/views/users.py +++ b/src/wuttaweb/views/users.py @@ -233,7 +233,7 @@ class UserView(MasterView): # pylint: disable=abstract-method session = self.Session() auth = self.app.get_auth_handler() - old_roles = set([role.uuid for role in user.roles]) + old_roles = {role.uuid for role in user.roles} new_roles = data["roles"] admin = auth.get_role_administrator(session)