sort and sort-by should retain meta

Description

sort and sort-by don't conceptually change the contents of their source. It would be useful if they retained meta while reordering their contents.

Patch: clj-2417.patch

Environment

None

Activity

Show:
Ghadi Shayban
October 21, 2018, 8:49 PM

Alex, you may have forgotten the patch

Alex Miller
October 22, 2018, 3:12 AM

yup

Gijs Stuurman
November 4, 2018, 1:49 PM

The original patch (clj-2417.patch) is missing the '(is ..' wrapping in the test, without which the test are run but failures not reported. clj-2417-fix-test.patch adds the missing wrapping as a separate patch. Another option is to fix this in the original patch and to re-apply it.

Alex Miller
November 4, 2018, 2:35 PM

The patch has already been applied, can you make a new ticket?

Gijs Stuurman
November 5, 2018, 5:47 PM
Completed

Assignee

Unassigned

Reporter

Alex Miller

Labels

Approval

Ok

Patch

Code and Test

Fix versions

Affects versions

Priority

Major
Configure