Opened 6 years ago

Closed 6 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, 6 years ago

Where can I find this function in the repository?

in reply to:  1 comment:2 by Baptiste Mispelon, 6 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, 6 years ago

Owner: changed from nobody to exploit19
Status: new → assigned

comment:4 by exploit19, 6 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, 6 years ago

Summary: Add function docstring to manage.py main function template → Add function docstring to manage.py main function template.
Triage Stage: Unreviewed → Accepted

comment:6 by Mariusz Felisiak, 6 years ago

Has patch: set

comment:7 by Mariusz Felisiak, 6 years ago

Version: 3.0 → master

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

Resolution: → fixed
Status: assigned → closed

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