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

[Enhancement Proposal] Create a pycodestyle checker action while push and merge #23

Open
LaoshuBaby opened this issue Jun 24, 2023 · 0 comments

Comments

@LaoshuBaby
Copy link
Collaborator

LaoshuBaby commented Jun 24, 2023

Current style error status:

b83548b at https://github.com/OSMChina/Keqing/tree/base-model-change :

(Hope there a tool that can convert pycodestyle output to markdown checklist or github code(with line position) jump link

keqing\__init__.py:82:80: E501 line too long (98 > 79 characters)
keqing\__init__.py:86:80: E501 line too long (125 > 79 characters)
keqing\__init__.py:100:80: E501 line too long (128 > 79 characters)
keqing\__init__.py:109:80: E501 line too long (130 > 79 characters)
keqing\__init__.py:113:80: E501 line too long (143 > 79 characters)
keqing\__init__.py:143:5: E303 too many blank lines (2)
keqing\__init__.py:232:80: E501 line too long (85 > 79 characters)
keqing\basic\model.py:46:80: E501 line too long (87 > 79 characters)
keqing\method\diff.py:10:23: E721 do not compare types, use 'isinstance()'
keqing\method\diff.py:13:25: E721 do not compare types, use 'isinstance()'
keqing\method\diff.py:16:25: E721 do not compare types, use 'isinstance()'
keqing\method\diff.py:19:28: E721 do not compare types, use 'isinstance()'
keqing\method\diff.py:22:28: E721 do not compare types, use 'isinstance()'
keqing\method\diff.py:25:28: E721 do not compare types, use 'isinstance()'
keqing\method\diff.py:28:31: E721 do not compare types, use 'isinstance()'
keqing\method\diff.py:31:31: E721 do not compare types, use 'isinstance()'
keqing\method\diff.py:52:80: E501 line too long (87 > 79 characters)
keqing\method\network.py:60:46: E711 comparison to None should be 'if cond is not None:'
keqing\method\network.py:75:80: E501 line too long (100 > 79 characters)
keqing\method\network.py:82:80: E501 line too long (81 > 79 characters)
keqing\method\query.py:13:80: E501 line too long (114 > 79 characters)
keqing\method\query.py:71:80: E501 line too long (80 > 79 characters)
keqing\plugin\nominatim\__init__.py:1:25: W292 no newline at end of file
keqing\plugin\normalizer\__init__.py:1:33: W292 no newline at end of file
keqing\plugin\normalizer\normalization_url.py:21:80: E501 line too long (85 > 79 characters)
keqing\plugin\overpass\__init__.py:2:32: W292 no newline at end of file
keqing\plugin\overpass\overpass_bbox.py:2:21: E712 comparison to True should be 'if cond is not True:' or 'if not cond:'
keqing\plugin\overpass\overpass_geocode.py:8:80: E501 line too long (106 > 79 characters)
keqing\type\constraint.py:15:80: E501 line too long (107 > 79 characters)
keqing\type\element.py:5:1: E303 too many blank lines (3)
keqing\type\element.py:28:80: E501 line too long (97 > 79 characters)
keqing\type\element.py:62:20: E711 comparison to None should be 'if cond is None:'
keqing\type\element.py:94:80: E501 line too long (97 > 79 characters)
keqing\type\element.py:138:80: E501 line too long (97 > 79 characters)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant