Opened 5 years ago

Closed 5 years ago

#31066 closed Cleanup/optimization (fixed)

Add function docstring to manage.py main function template.

Reported by: Antonio Vazquez Owned by: exploit19
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: yes UI/UX: no

Description

When creating a django project scaffold, manage.py is missing a function docstring in main(). Please add a basic function docstring to the manage.py skeleton in the django project scaffold, to keep from getting lint errors on new projects.

Change History (8)

comment:1 by Bruce, 5 years ago

Where can I find this function in the repository?

in reply to:  1 comment:2 by Baptiste Mispelon, 5 years ago

Replying to Bruce:

Where can I find this function in the repository?

It's in the django/conf/project_template/ directory: https://github.com/django/django/blob/master/django/conf/project_template/manage.py-tpl

comment:3 by exploit19, 5 years ago

Owner: changed from nobody to exploit19
Status: newassigned

comment:4 by exploit19, 5 years ago

I've created a PR to add a docstring. Please review and let me know if any change required. Thanks!
https://github.com/django/django/pull/12188

comment:5 by Mariusz Felisiak, 5 years ago

Summary: Add function docstring to manage.py main function templateAdd function docstring to manage.py main function template.
Triage Stage: UnreviewedAccepted

comment:6 by Mariusz Felisiak, 5 years ago

Has patch: set

comment:7 by Mariusz Felisiak, 5 years ago

Version: 3.0master

comment:8 by Mariusz Felisiak <felisiak.mariusz@…>, 5 years ago

Resolution: fixed
Status: assignedclosed

In d8ab2c9:

Fixed #31066 -- Added a docstring to main() function in manage.py template.

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