From ec982fe1689bfcc7a1483a03237f5eb583cd3f9b Mon Sep 17 00:00:00 2001 From: Lance Edgar Date: Mon, 1 Sep 2025 10:22:29 -0500 Subject: [PATCH] fix: fix 'consider-using-set-comprehension' for pylint --- .pylintrc | 1 - src/wuttaweb/forms/schema.py | 2 +- src/wuttaweb/views/users.py | 2 +- 3 files changed, 2 insertions(+), 3 deletions(-) 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)