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

Catch app leak during convert #51

Open
radiac opened this issue Dec 7, 2024 · 0 comments
Open

Catch app leak during convert #51

radiac opened this issue Dec 7, 2024 · 0 comments

Comments

@radiac
Copy link
Owner

radiac commented Dec 7, 2024

In several scenarios I've seen an app instance reference be carried across during a nanodjango convert operation. This really isn't desirable - nanodjango shouldn't be anywhere near a full Django project, so it's a sign that nanodjango's converter didn't understand what was happening.

The conversion process is complicated so this sort of thing is to be expected, but it's going to cause some weird and wonderful errors.

Detect when Django is being instantiated after a convert, and show a warning that something looks wrong and will need to be fixed manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

1 participant