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

Fix or skip deepsource check in commands.py #341

Closed
wants to merge 34 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
18e0d0a
Fix PYL-W0612: Unused variable found
winebarrel Oct 15, 2023
bc31ade
skipcq PYL-W0402
winebarrel Oct 15, 2023
8864623
Fix PYL-W1505: Method used has been deprecated
winebarrel Oct 15, 2023
909e168
Fix PYL-R1714: Consider using "in"
winebarrel Oct 15, 2023
d9ef87a
Fix PYL-W1201: Logging is not lazy
winebarrel Oct 15, 2023
a1b7f63
Fix PYL-R1722: Use of "exit()" or "quit()" detected
winebarrel Oct 15, 2023
95c313b
Fix PTC-W0048: "if" statements can be merged
winebarrel Oct 15, 2023
e98794c
skipcq PTC-W0065
winebarrel Oct 15, 2023
3eb862f
Fix FLK-E127: Continuation line over-indented for visual indent
winebarrel Oct 15, 2023
74f6d56
Fix FLK-E231: Missing whitespace after ",", ";", or ":"
winebarrel Oct 15, 2023
e34d959
Fix PYL-R0123: Comparing to literal
winebarrel Oct 15, 2023
56322c3
Fix FLK-E231: Missing whitespace after ",", ";", or ":"
winebarrel Oct 15, 2023
ea64c13
Fix FLK-E251: Unexpected spaces around keyword / parameter equals
winebarrel Oct 15, 2023
4199b00
Fix FLK-E261: At least two spaces before inline comment
winebarrel Oct 15, 2023
2c8b65d
Fix FLK-E262: Inline comment should start with "#"
winebarrel Oct 15, 2023
368ed54
Fix FLK-E302: Expected 2 blank lines
winebarrel Oct 15, 2023
ecddf70
Fix FLK-E265: Block comment should start with "#"
winebarrel Oct 15, 2023
63738d4
Fix FLK-E302: Expected 2 blank lines
winebarrel Oct 15, 2023
ce1801a
Fix FLK-E303: Too many blank lines found
winebarrel Oct 15, 2023
574192a
Fix FLK-E303: Too many blank lines found
winebarrel Oct 15, 2023
c02e4e9
Fix FLK-E501: Line too long
winebarrel Oct 15, 2023
2e890ee
Fix FLK-D300: Use """triple double quotes"""
winebarrel Oct 15, 2023
bc57813
Fix PYL-C0325: Unnecessary parentheses after keyword
winebarrel Oct 15, 2023
2265a60
skiqcq PYL-W0511
winebarrel Oct 15, 2023
e85e51d
Fix FLK-D200: One-line docstring should fit on one line with quotes
winebarrel Oct 15, 2023
fa2af39
skipcq PY-D0003
winebarrel Oct 15, 2023
febd092
Fix PYL-W0105: Unassigned string statement
winebarrel Oct 15, 2023
3c6b523
skipcq PY-D0002
winebarrel Oct 15, 2023
7a924c2
Fix FLK-D202: No blank lines allowed after function docstring
winebarrel Oct 15, 2023
466a392
Fix FLK-D211: No blank lines allowed before class docstring
winebarrel Oct 15, 2023
5bf1ad4
skipcq PYL-C0209
winebarrel Oct 15, 2023
f670a54
Fix PYL-W1202: Lazy formatting of message string passed to logging mo…
winebarrel Oct 15, 2023
aab8ba2
skiqcq PY-R1000
winebarrel Oct 15, 2023
bbed02a
Fix syntax error
winebarrel Oct 15, 2023
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
Prev Previous commit
Fix syntax error
winebarrel committed Oct 15, 2023
commit bbed02af80492d1a979a00522af2009e9628d6b9
2 changes: 1 addition & 1 deletion qds_sdk/commands.py
Original file line number Diff line number Diff line change
@@ -149,7 +149,7 @@ def run(cls, **kwargs):

while not Command.is_done(cmd.status):
if sighandler.received_term_signal:
logging.warning("Received signal %s. Canceling Qubole Command ID: %s" sighandler.last_signal, cmd.id)
logging.warning("Received signal %s. Canceling Qubole Command ID: %s", sighandler.last_signal, cmd.id)
cls.cancel(cmd)
sys.exit()
time.sleep(Qubole.poll_interval)