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

Attachments

2

Activity

Show:

Gijs StuurmanNovember 5, 2018 at 5:47 PM

Alex MillerNovember 4, 2018 at 2:35 PM

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

Gijs StuurmanNovember 4, 2018 at 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 MillerOctober 22, 2018 at 3:12 AM

yup

Ghadi ShaybanOctober 21, 2018 at 8:49 PM

Alex, you may have forgotten the patch

Completed

Details

Assignee

Reporter

Labels

Approval

Ok

Patch

Code and Test

Priority

Affects versions

Fix versions

Created October 19, 2018 at 4:43 PM
Updated November 6, 2018 at 12:33 PM
Resolved November 6, 2018 at 12:33 PM