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

Add support for excluding south migrations #12

Closed
nijel opened this issue Feb 1, 2014 · 3 comments
Closed

Add support for excluding south migrations #12

nijel opened this issue Feb 1, 2014 · 3 comments

Comments

@nijel
Copy link

nijel commented Feb 1, 2014

While using South and it's generated migrations, you'll get lot of automatically generated code with poor PEP-8 compliance, so it would be great to ignore them.

It could probably be done automatically once using Django (ignore */migrations/*.py) or providing some generic way to define files to be skipped from checking.

@carlio
Copy link
Member

carlio commented Feb 26, 2014

Hi @nijel - this should already be happening, and I can't reproduce the problem with the current master branch. However I have made several changes to the ignoring code recently so probably I have accidentally fixed the issue.

Is the repository you're using publicly available? If not, could you try using the current master branch and see if you still have this issue? Thanks!

@nijel
Copy link
Author

nijel commented Feb 26, 2014

The code is hosted at github - https://github.com/nijel/weblate

@carlio
Copy link
Member

carlio commented Feb 26, 2014

Great thanks, I'll take a look. Actually I think I just found the problem, it was not showing up for me due to a different bug!

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

No branches or pull requests

2 participants