Opened 3 years ago
Closed 3 years ago
#32621 closed Cleanup/optimization (wontfix)
class names might follow pep8
| Reported by: | BeryCZ | Owned by: | nobody |
|---|---|---|---|
| Component: | Core (Other) | Version: | 3.2 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description (last modified by )
Hi,
I've noticed that some class names don't follow pep 8 and it's quite annoying to either remember or search for the right name.
What comes to mind rn is JsonResponse (which is ok) and classes like JSONField, JSONCatalog, JSONString and some more.
Would be nice to change the class names some time to Json and for backward compatibility to do JSONField = JsonField
Change History (3)
comment:1 by , 3 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 3 years ago
| Description: | modified (diff) |
|---|
comment:3 by , 3 years ago
| Component: | Uncategorized → Core (Other) |
|---|---|
| Resolution: | → wontfix |
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Thanks for this proposition.
I would rather do the opposite, as suggested in #22895 and as we did for a new API, e.g.
ASGIRequest,ASGIHandler. Nonetheless, it's not worth to change an existing API.