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

DP-5290 python-sdk/cdn: 查询域名带宽，支持type参数 #456

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

DP-5290 python-sdk/cdn: 查询域名带宽，支持type参数

f3a0b9b
Select commit
Loading
Failed to load commit list.
Open

DP-5290 python-sdk/cdn: 查询域名带宽，支持type参数 #456

DP-5290 python-sdk/cdn: 查询域名带宽，支持type参数
f3a0b9b
Select commit
Loading
Failed to load commit list.
qiniu-x / pylint completed Dec 9, 2024 in 5h 47m 58s

pylint found 12 issues related to your changes

This is the detailed log.

If you have any questions about this comment, feel free to raise an issue here.

Details

🔍 Check Results Details

The following shows all issues found related to your changes:

qiniu/services/cdn/manager.py:12: Missing class docstring (missing-class-docstring)
qiniu/services/cdn/manager.py:20: Missing function or method docstring (missing-function-docstring)
qiniu/services/cdn/manager.py:20: Redefining built-in 'str' (redefined-builtin)
qiniu/services/cdn/manager.py:21: Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return)
qiniu/services/cdn/manager.py:20: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
qiniu/services/cdn/manager.py:97: Formatting a regular string which could be an f-string (consider-using-f-string)
qiniu/services/cdn/manager.py:100: Too many arguments (6/5) (too-many-arguments)
qiniu/services/cdn/manager.py:124: Formatting a regular string which could be an f-string (consider-using-f-string)
qiniu/services/cdn/manager.py:127: Too many arguments (6/5) (too-many-arguments)
qiniu/services/cdn/manager.py:151: Formatting a regular string which could be an f-string (consider-using-f-string)
qiniu/services/cdn/manager.py:8: standard import "enum.Enum" should be placed before first party imports "qiniu.http", "qiniu.compat.is_py2", "qiniu.compat.is_py3"  (wrong-import-order)
qiniu/services/cdn/manager.py:10: standard import "hashlib" should be placed before first party imports "qiniu.http", "qiniu.compat.is_py2", "qiniu.compat.is_py3"  (wrong-import-order)

🔄 How to Handle?

  1. Fix the issues above and submit your code again
  2. Or click the Re-run button to run the check again
  3. If you think this is a false positive, please contact your support team

ℹ️ Notes

  • To avoid too many comments, only the top 10 issues will be shown in PR comments
  • For any other issues, feel free to create an issue in reviewbot repository
View more details on qiniu-x