search

hhatto / autopep8

A tool that automatically formats Python code to conform to the PEP 8 style guide.
https://pypi.org/project/autopep8/
MIT License
4.54k stars 291 forks source link

Version dependent operator whitespace function. #696

Closed keriksson-rosenqvist closed 11 months ago

keriksson-rosenqvist commented 11 months ago

Solve a rename issue in pycodestyle.

codecov-commenter commented 11 months ago

Codecov Report

Patch coverage: 75.00% and project coverage change: -0.04% :warning:

Comparison is base (5b9110b) 97.83% compared to head (36941c7) 97.79%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #696 +/- ## ========================================== - Coverage 97.83% 97.79% -0.04% ========================================== Files 1 1 Lines 2398 2402 +4 ========================================== + Hits 2346 2349 +3 - Misses 52 53 +1 ``` | [Files Changed](https://app.codecov.io/gh/hhatto/autopep8/pull/696?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Hideo+Hattori) | Coverage Δ | | |---|---|---| | [autopep8.py](https://app.codecov.io/gh/hhatto/autopep8/pull/696?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Hideo+Hattori#diff-YXV0b3BlcDgucHk=) | `97.79% <75.00%> (-0.04%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

keriksson-rosenqvist commented 11 months ago

The missed line in the diff coverage is what's used when running a version of Python that is not supported by pycodestyle >= 2.11.0. It is used in the Python package/ build (3.7) action.

RmaxTwice commented 11 months ago

We really need this PR merged to avoid issues even with pre commit hooks new installations...

shaperilio commented 11 months ago

699, #696, #691, #690 are all the same. I suggest someone look into this comment and push their PR forward with a solution.

ryanpetrello commented 11 months ago

@hhatto any thoughts or feedback on this? This issue is preventing adoption of the latest autopep8 on a project I maintain 👍

ryanpetrello commented 11 months ago

@hhatto any chance you'd be willing to release a new version of autopep8 to PyPI?

🙏

(thank you!)

hhatto commented 11 months ago

@ryanpetrello autopep8 version 2.0.3 has been released 🚀 https://pypi.org/project/autopep8/2.0.3/

ryanpetrello commented 11 months ago

Thank you @hhatto 🙏