Thank you for picking this up @salvo-polizzi

This is missing doc updates 👍

Hi @sarahboyce, do you think it would be better to add this as a note in the shell section of django-admin reference, or were you thinking of something else?

do you think it would be better to add this as a note in the shell section of django-admin reference, or were you thinking of something else?

Basically all docs that were updated in the previous commit refer to the models as the only thing auto imported. All of these references would need updating

Hi @sarahboyce,

Thank you for your review! 👍

I've made some updates to the documentation:

• Release Notes (6.0): Added a brief release note.
• custom_shell.txt: Clarified that, in addition to models, the default imports are also mentioned.
• Django-admin.txt: Included a .. versionchanged:: 6.0 note listing the default imports.

Please let me know if I missed anything or if there’s anything you'd like me to refine.

@salvo-polizzi just so you are aware of my current thoughts, I am reluctant to merge anything until at least 2 months after the 5.2 final release. I would like us to receive some user feedback on the auto imports before we build on top of it

@salvo-polizzi just so you are aware of my current thoughts, I am reluctant to merge anything until at least 2 months after the 5.2 final release. I would like us to receive some user feedback on the auto imports before we build on top of it

Sure, let's keep this for later discussions when we have a greater consensus and feedback about this.

Note we did an update in de1117e which added tests.shell.tests.ShellCommandTestCase.test_no_settings, I think it's worth adding an assert to confirm that these are not imported to the namespace

Thank you @sarahboyce for pointing out this 👍

Hi @sarahboyce, thank you for the review ⭐ (It's good to hear from you)!