Optional auto adding namespace the functions used in the code and reverse_code arguments of RunPython about django-squash HOT 6 CLOSED

Inside the extract_function(), it gets called here.

Please add an option to turn function renaming on/off. In case someone wants to opt out.

Lastly. Please add tests. I can't stress this last one enough.

from django-squash.

That makes sense, PRs are welcome :D

from django-squash.

@kingbuzzman Since you know the codebase, can you please point me to the relevant parts of the source code that I should study in order to change and implement the proposed feature? This will probably save me some debug time....

from django-squash.

@luzfcb I really want to close this issue, but didnt want to leave you hanging either.

I started https://github.com/kingbuzzman/django-squash/pull/42/files to address this, im not sold on the name DJANGO_SQUASH_CUSTOM_RENAME_FUNCTION -- but its a working draft.

The idea is that you define a function somewhere in your code, and add DJANGO_SQUASH_CUSTOM_RENAME_FUNCTION='dot.path.location.where.is.the.function' in your settings.py and implement it obviously.. then this function would be called every time the squash finds an non-elidable operation and you can there dictate the name of the function. Still need to write an integration test for it.

But, does this work? is this what you had in mind? So far here are the variables you would have available migration (where the function is), name (name of function), app (name of the app), operation (operation object). From there you can name the functinon as you see fit.

Please let me know what you think. Thanks

from django-squash.