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

Fix incorrect method #691

Closed
wants to merge 1 commit into from
Closed

Fix incorrect method #691

wants to merge 1 commit into from

Conversation

LordRatte
Copy link

No description provided.

@LordRatte
Copy link
Author

  File "/opt/homebrew/lib/python3.11/site-packages/autopep8.py", line 761, in fix_e225
    pycodestyle.missing_whitespace_around_operator(fixed, ts))
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'pycodestyle' has no attribute 'missing_whitespace_around_operator'. Did you mean: 'whitespace_around_operator'?

@shaperilio
Copy link

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

@LordRatte
Copy link
Author

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

What do you mean "Push their solution forward"? Is that a process on this project or a new git thing etc.?

@shaperilio
Copy link

@LordRatte I just mean that there are several PRs that claim to make the exact same fix, but I think that my comment should be addressed. If someone does that (i.e. put in a bit more work), maybe their PR will be chosen as the fix, or at least a starting point.

(I'm just another random person and cannot merge any PRs; just making a suggestion in how to get things ready for merging.)

@hhatto
Copy link
Owner

hhatto commented Aug 25, 2023

thanks for contribution.
This has been fixed in pull-request #696, so we will close this one 🙏
We will release a new version within a few days.

@hhatto hhatto closed this Aug 25, 2023
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

Successfully merging this pull request may close these issues.

3 participants