-
-
Notifications
You must be signed in to change notification settings - Fork 308
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
Update pep8 #91
Comments
In GitLab by @sigmavirus24 on Mar 25, 2015, 09:50 Please check existing (closed and open) issues before filing new ones. We're specifically pinning to <1.6.0 because it breaks other features. If you want to use newer (broken) versions of pep8, pin to flake8 2.3.0 |
In GitLab by @rfleschenberg on Mar 26, 2015, 02:54 I searched the existing issues, but could not find a related one. Can you point me to it? So, you're "specifically pinning to" what? A certain pep8 version, I suppose? I realize that. The point is that the pep8 version you are using is buggy. It contradicts PEP 8. You should either update your pep8 version or backport the fix for this bug. |
In GitLab by @rfleschenberg on Mar 26, 2015, 02:59 OK. Now I read a little more on the other issues and realize how they are related. Sorry. I was also thinking you had actually bundled pep8, but it seems I was looking at the wrong source? Anyway, closing. I hope this gets fixed soon. |
@asottile Thanks! @sigmavirus24, I want to apologize for my comments. Reading them again now after 6 years have passed, I realize I sounded like an entitled asshole :( Sorry. |
Appreciate it @rfleschenberg We all have bad moments. I've definitely had mine and I'm trying to avoid having more by trying to figure out my stress coping strategies right now. |
In GitLab by @rfleschenberg on Mar 25, 2015, 07:10
The pep8 tool that comes bundled with flake8 is outdated and reports false warnings. You should update.
E.g.:
The text was updated successfully, but these errors were encountered: