Opened 4 years ago

Closed 4 years ago

#31428 closed Cleanup/optimization (fixed)

Allow empty message in management command stdout and stderr proxies.

Reported by: François Freitag Owned by: François Freitag
Component: Core (Management commands) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Django management commands wrap stdout and stderr in an OutputWrapper that adds a \n at the end of the text provided as the out argument.

I suggest allowing self.stdout.write() and self.stderr.write() to add a newline to respectively stdout and stderr. Currently, it fails because msg is a positional argument.

​PR

Change History (2)

comment:1 by Mariusz Felisiak, 4 years ago

Owner: changed from nobody to François Freitag
Status: new → assigned
Summary: Allow empty message in management command stdout and stderr proxies → Allow empty message in management command stdout and stderr proxies.
Triage Stage: Unreviewed → Accepted
Type: New feature → Cleanup/optimization

I would rather focus on #21429, but we can accept this small cleanup.

comment:2 by Mariusz Felisiak <felisiak.mariusz@…>, 4 years ago

Resolution: → fixed
Status: assigned → closed

In e03eb8db:

Fixed #31428 -- Allowed empty message in management command self.stdout/err proxies.

Note: See TracTickets for help on using tickets.
Back to Top