-
Notifications
You must be signed in to change notification settings - Fork 117
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
Errors on names of migrations #167
Comments
okay, i'm getting other errors as well that should be suppressed by pylint-django in my understanding, e.g. using pylint 1.9.2, pylint-django 0.11.1, python 3.6.5, on windows. any help would be appreciated. |
So the first error is because module name starts with a digit, not a letter (if I read the regexp correctly). Can you post your command line by which you invoke pylint and also the directory structure around your migrations. I'm not getting such errors for one of my Django projects and we do have lots of migrations. |
it's folder structure see https://github.com/fsr-itse/EvaP/, or see this screenshot |
this isn't invoking pylint-django. If this is your command line then pylint is working as expected b/c you are not enabling the pylint-django plugin. |
as i said i have enabled the plugin in my .pylintrc. I get the very same errors when using |
Well then post your |
i have narrowed it down to this: the default |
To summarize: when |
yes, thank you. sorry for the cumbersome communication with me. |
Either a bug in pylint or desired behavior. Waiting for info: |
@PCManticore can you comment on the above comment/email thread ? |
@atodorov this is a bit of an undefined behaviour. There's currently no guarantee for the plugins to get the fully loaded configuration file at the time they are registered. But you might check here if moving the plugin loading after the configuration loading would help you, but I don't know if that can have any side effects or not. |
@karyon can you reproduce with the latest version ? It looks like this may have been fixed meanwhile. |
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 not report anything for the migrations in that app, regardless of whether an i guess that's the intended behavior, always ignoring the migration directories? if that's the case, i would feature request the option to disable that. migrations can contain considerable amount of code as well (especially when using RunPython), and I think it's wrong to not get any pylint warnings in that code. |
I am also seeing this issue
I solved this by adding
|
I'm getting lots of warnings on my migrations, e.g.:
if this code here is supposed to prevent any errors on migrations error, it doesn't work :) if it's not supposed to do that, i would feature request that these migrations names are allowed by default.
The text was updated successfully, but these errors were encountered: