Johannes Linke

Results 17 comments of Johannes Linke

no idea whether other fixes are needed. i cannot provide a pull request at least until next year.

okay, i'm getting other errors as well that should be suppressed by pylint-django in my understanding, e.g. `Constant name "app_name" doesn't conform to UPPER_CASE naming style`. however, i have `pylint_django`...

it's `pylint.exe evap`. folder structure see https://github.com/fsr-itse/EvaP/, or see this [screenshot](https://user-images.githubusercontent.com/1891915/41832366-0c3d4d1a-784c-11e8-95a3-4941c509e9a1.png)

as i said i have enabled the plugin in my .pylintrc. I get the very same errors when using `--load-plugins pylint_django`.

i have narrowed it down to this: the default `.pylintrc` file contains `ignore=CVS`, and with that i get the erroneous warnings. when removing that part, i do not get these...

yes, thank you. sorry for the cumbersome communication with me.

I cannot say conclusively, because pylint crashes before finishing the analysis, but it seems it is completing analysis of the app which contains those migrations before crashing, and it does...

duplicate of #89 and documented in #91, although i would definitely appreciate the fix you provide in your PR.

Right. I didn't fix the entire bug, only those (few) cases where the line would actually fit into the width limit. See my comment here: https://github.com/rust-lang/rustfmt/issues/4968#issuecomment-949461340

@ChinYing-Li maybe you could push the branch again to re-trigger the CI so it might become green? Other than that, since 1.4.37 has been released a while ago... :)