Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[8.0] fix: new pylint version fixes #7810

Merged
merged 1 commit into from
Oct 2, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion release.notes
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ FIX: (#7787) added a 30s gfal2 timeout for downloading the SRR

FIX: (#7741) RequestTaskAgent only considers requests in final states, and consider files in intermediate state as problematic (https://github.com/DIRACGrid/DIRAC/issues/7116)
NEW: (#7741) RequestTaskAgent uses getBulkRequestStatus instead of getRequestStatus
RMS: (#7741)
RMS: (#7741)
NEW: (#7741) implement getRequestStatus

[v8.0.52]
Expand Down
4 changes: 2 additions & 2 deletions src/DIRAC/Core/Tornado/Server/HandlerManager.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"""
from tornado.web import RequestHandler

from DIRAC import gConfig, gLogger, S_ERROR, S_OK
from DIRAC import S_ERROR, S_OK, gConfig, gLogger
from DIRAC.ConfigurationSystem.Client import PathFinder
from DIRAC.Core.Base.private.ModuleLoader import ModuleLoader

Expand Down Expand Up @@ -123,7 +123,7 @@ def __load(self, instances, componentType, pathFinder):
# see DIRAC.Core.Tornado.Server.private.BaseRequestHandler for more details
# this method should return a list of routes associated with the handler, it is a regular expressions
# see https://www.tornadoweb.org/en/stable/routing.html#tornado.routing.URLSpec, ``pattern`` argument.
urls = handler._BaseRequestHandler__pre_initialize()
urls = handler._BaseRequestHandler__pre_initialize() # pylint: disable=no-member

# First of all check if we can find route
if not urls:
Expand Down
2 changes: 1 addition & 1 deletion src/DIRAC/Core/Tornado/Server/TornadoServer.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

import tornado.iostream

tornado.iostream.SSLIOStream.configure(
tornado.iostream.SSLIOStream.configure( # pylint: disable=no-member
"tornado_m2crypto.m2iostream.M2IOStream"
) # pylint: disable=wrong-import-position

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -953,7 +953,7 @@ async def __execute(self, *args, **kwargs): # pylint: disable=arguments-differ
# you need to define the finish_<methodName> method.
# This method will be started after _executeMethod is completed.
elif callable(finishFunc := getattr(self, f"finish_{self.__methodName}", None)):
finishFunc()
finishFunc() # pylint: disable=not-callable

# In case nothing is returned
elif self.__result is None:
Expand Down
Loading