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.

Sign up for GitHub

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

Jump to bottom

bad-str-strip-call errors in pylint #6002

Closed
ThePhisch opened this issue Nov 25, 2022 · 0 comments
Closed

bad-str-strip-call errors in pylint #6002

ThePhisch opened this issue Nov 25, 2022 · 0 comments
Assignees
@ThePhisch
Labels
bug Protocols
Milestone
1.30.0

Comments

@ThePhisch
Copy link
Contributor

Description

Pylint found 3 occurrences of E1310 bad-str-strip-call in the codebase.

Steps to reproduce

  1. run donkeyrider with -d centos7 -s syntax --build
  2. examine output of running pylint

Rucio Version

No response

Additional Information

No response
@ThePhisch ThePhisch added the bug label Nov 25, 2022
@bari12 bari12 closed this as completed in 4d8d26c Nov 25, 2022
bari12 added a commit that referenced this issue Nov 25, 2022
@bari12
Merge pull request #6003 from ThePhisch/patch-6002-fix_small_pylint
c0f895d 
Protocols: Fix bad-str-strip-call (Pylint); Fix #6002
@bari12 bari12 added this to the 1.30.0 "The Donkeynator" milestone Nov 25, 2022
ThePhisch added a commit to ThePhisch/rucio that referenced this issue Dec 19, 2022
@ThePhisch
Protocols: Fix bad-str-strip-call (Pylint); Fix rucio#6002
d87e89a 
The replacement is semantically equivalent. Only spaces (but any amount
of these) will be stripped.
ThePhisch added a commit to ThePhisch/rucio that referenced this issue Dec 19, 2022
@ThePhisch
Protocols: Fix bad-str-strip-call (Pylint); Fix rucio#6002
0c85423 
The replacement is semantically equivalent. Only spaces (but any amount
of these) will be stripped.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ThePhisch ThePhisch

Labels
bug Protocols
Projects
None yet
Milestone
1.30.0
Development

No branches or pull requests
2 participants
@bari12 @ThePhisch