Opened 7 years ago

Closed 7 years ago

#28152 closed Cleanup/optimization (fixed)

Change SetSerializer to serialize to set literals

Reported by: Jon Dufresne Owned by: Jon Dufresne
Component: Migrations 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

While looking for all code that could be converted to set literals and set comprehension, it was noticed that the migration's SetSerializer does not use set literals. If we're to convert all internal code to user set literals, the auto generated code should follow the same convention.

Idea originally discussed in ​PR.

Change History (3)

comment:1 by Jon Dufresne, 7 years ago

Has patch: set
Owner: changed from nobody to Jon Dufresne
Status: new → assigned

​PR

comment:2 by Tim Graham, 7 years ago

Triage Stage: Unreviewed → Accepted

comment:3 by Tim Graham <timograham@…>, 7 years ago

Resolution: → fixed
Status: assigned → closed

In f599747:

Fixed #28152 -- Made migrations serialize sets as set literals rather than set().

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