Project

General

Profile

Cleanup #40992

cephfs-shell: Multiple flake8 errors

Added by Varsha Rao 27 days ago. Updated 27 days ago.

Status:
Pending Backport
Priority:
Normal
Assignee:
Category:
Code Hygiene
Target version:
Start date:
Due date:
% Done:

0%

Tags:
Backport:
nautilus
Reviewed:
Affected Versions:
Component(FS):
cephfs-shell
Labels (FS):
Pull request ID:

Description

After ignoring E501 and W503 flake8 errors, following needs to be fixed:

(venv) [root@localhost ceph]# flake8 --ignore=E501,W503 src/tools/cephfs/cephfs-shell
src/tools/cephfs/cephfs-shell:80:28: E241 multiple spaces after ':'
src/tools/cephfs/cephfs-shell:171:47: E226 missing whitespace around arithmetic operator
src/tools/cephfs/cephfs-shell:770:64: W504 line break after binary operator
src/tools/cephfs/cephfs-shell:771:37: W504 line break after binary operator
src/tools/cephfs/cephfs-shell:772:36: W504 line break after binary operator
src/tools/cephfs/cephfs-shell:776:57: W504 line break after binary operator
src/tools/cephfs/cephfs-shell:777:54: W504 line break after binary operator
src/tools/cephfs/cephfs-shell:778:40: W504 line break after binary operator
src/tools/cephfs/cephfs-shell:779:39: W504 line break after binary operator
src/tools/cephfs/cephfs-shell:1070:48: E226 missing whitespace around arithmetic operator
src/tools/cephfs/cephfs-shell:1074:40: E226 missing whitespace around arithmetic operator
src/tools/cephfs/cephfs-shell:1226:37: E226 missing whitespace around arithmetic operator
src/tools/cephfs/cephfs-shell:1242:19: E126 continuation line over-indented for hanging indent
src/tools/cephfs/cephfs-shell:1289:44: E226 missing whitespace around arithmetic operator

E501 line too long
W503 line break before binary operator


Related issues

Copied to fs - Backport #41089: nautilus: cephfs-shell: Multiple flake8 errors In Progress

History

#1 Updated by Varsha Rao 27 days ago

  • Status changed from New to Need Review
  • Pull request ID set to 29374

#2 Updated by Varsha Rao 27 days ago

Not ignoring E501 instead limiting line length to 100.

cephfs-shell:80:28: E241 multiple spaces after ':'
cephfs-shell:171:47: E226 missing whitespace around arithmetic operator
cephfs-shell:185:101: E501 line too long (101 > 100 characters)
cephfs-shell:770:64: W504 line break after binary operator
cephfs-shell:771:37: W504 line break after binary operator
cephfs-shell:772:36: W504 line break after binary operator
cephfs-shell:776:57: W504 line break after binary operator
cephfs-shell:777:54: W504 line break after binary operator
cephfs-shell:778:40: W504 line break after binary operator
cephfs-shell:779:39: W504 line break after binary operator
cephfs-shell:1070:48: E226 missing whitespace around arithmetic operator
cephfs-shell:1074:40: E226 missing whitespace around arithmetic operator
cephfs-shell:1226:37: E226 missing whitespace around arithmetic operator
cephfs-shell:1242:19: E126 continuation line over-indented for hanging indent
cephfs-shell:1289:44: E226 missing whitespace around arithmetic operator

#3 Updated by Patrick Donnelly 27 days ago

  • Status changed from Need Review to Pending Backport

#4 Updated by Patrick Donnelly 20 days ago

  • Copied to Backport #41089: nautilus: cephfs-shell: Multiple flake8 errors added

Also available in: Atom PDF