Since the warning is to stderr, it doesn't break the main usecase of doing "python manage.py inspectdb > models.py"
Django use comment to tell that there was an error when the table introspection failed, but it should be a with a detailed traceback in my opinion. But it looks like it's me who have an unpopular opinion so I will implement the PR with a comment instead of a warning. Le mar. 1 déc. 2020 à 16:18, Tim Graham <timogra...@gmail.com> a écrit : > Please try to make it easy to understand the issue without looking at > tickets and pull requests. Even after I did that, I couldn't find why you > believe it should be a warning and not a comment. Why is this situation > different from all the other places Django uses comments to indicate where > introspection may not be perfect? > > Wouldn't a warning break the documented usage of inspectdb because there > would be non-Python text in models.py from the warning output? > > $ python manage.py inspectdb > models.py > > On Tuesday, December 1, 2020 at 9:12:53 AM UTC-5 damio wrote: > >> Hi, >> >> I got some feedback on the ticket that I should use comments to display >> the warning instead of stderr, since I don't really agree with that, I'm >> requesting more feedback to know which way I should do it. >> >> Ticket: https://code.djangoproject.com/ticket/32234 >> PR: https://github.com/django/django/pull/13736 >> >> Thanks, >> Damien >> > -- > You received this message because you are subscribed to a topic in the > Google Groups "Django developers (Contributions to Django itself)" group. > To unsubscribe from this topic, visit > https://groups.google.com/d/topic/django-developers/NhhpjqeQ8MQ/unsubscribe > . > To unsubscribe from this group and all its topics, send an email to > django-developers+unsubscr...@googlegroups.com. > To view this discussion on the web visit > https://groups.google.com/d/msgid/django-developers/b455bb92-6b91-4f2f-bdaa-8932d3cb2893n%40googlegroups.com > <https://groups.google.com/d/msgid/django-developers/b455bb92-6b91-4f2f-bdaa-8932d3cb2893n%40googlegroups.com?utm_medium=email&utm_source=footer> > . > -- You received this message because you are subscribed to the Google Groups "Django developers (Contributions to Django itself)" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-developers+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-developers/CAAQ5yigGWu8%3D5L5_EXS3kc99584YtiYfb2SN4eRyJOkdR7h8iQ%40mail.gmail.com.
