Skip to content

Commit 141f2e4

Browse files
committed
ruff/ comment change
1 parent 15d09e8 commit 141f2e4

2 files changed

Lines changed: 15 additions & 9 deletions

File tree

irods/access.py

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -204,24 +204,27 @@ def __repr__(self):
204204

205205
class iRODSAccess(_iRODSAccess_base, metaclass=_Access_LookupMeta):
206206
"""
207-
Represents an ACL in iRODS. An instance of this class functions as a data container
208-
to convey information to the iRODS server (in the `set` call) and back again to the client
209-
again (in the `get` call).
207+
Represents an ACL in iRODS.
208+
209+
An instance of this class functions as a data container to convey information to the iRODS
210+
server (in the `set` call) and back again to the client again (in the `get` call).
210211
"""
211212

212213
def __init__(self, access_name, path, user_name="", user_zone="", user_type=None):
214+
# noqa: D207
213215
self.codes = self.__class__.codes
214216
self.strings = self.__class__.strings
215217
super().__init__(access_name, path, user_name, user_zone, user_type)
216218

217219

218220
class ACLOperation(iRODSAccess):
219221
"""
220-
Similar to its base class iRODSAccess, this class represents an ACL to be set on an object.
221-
but this class is the counterpart used for the atomic ACLs api. It differs from its base
222-
class in that it has no field to store a logical object path. (For an atomic API call, i
223-
here is always a single logical path to which all operations apply, meaning that it is
224-
appropriate to conveyed that in a location separate from the operations themselves.)
222+
Represents an operation to be performed in iRODS' atomic ACL api.
223+
224+
Similar to its base class, iRODSAccess, this class names an ACL to be set on an object.
225+
It differs, however, in that it forgoes option to store a logical object path. (In the atomic
226+
API call, there is always a single logical path to which all such operations apply, thus
227+
it is appropriate that the path parameter is in a location separate from the operations.)
225228
""" # noqa: D400
226229

227230
def __init__(self, access_name: str, user_name: str = "", user_zone: str = ""):
@@ -233,6 +236,7 @@ def __init__(self, access_name: str, user_name: str = "", user_zone: str = ""):
233236
)
234237

235238
def __eq__(self, other):
239+
# noqa: D105
236240
return (
237241
self.access_name,
238242
self.user_name,
@@ -244,6 +248,8 @@ def __eq__(self, other):
244248
)
245249

246250
def __hash__(self):
251+
# noqa: D105
252+
247253
# Hash in a way consistent with an iRODSAccess having path "".
248254
return hash((
249255
self.access_name,

irods/exception.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -650,7 +650,7 @@ class SYS_INVALID_INPUT_PARAM(SystemException):
650650
code = -130000
651651

652652

653-
class SYS_INTERNAL_ERR(SystemException):
653+
class SYS_INTERNAL_ERR(SystemException): # noqa: N801
654654
code = -154000
655655

656656

0 commit comments

Comments
 (0)