sort and sort-by should retain meta

Question

Port CLJ-2417 to ClojureScript.

Answer 1

Comment made by: namenu

My first contribution to CLJS.

And yes I've submitted CA and passed tests.

Answer 2

Comment made by: mfikes

Thanks for the contribution!

Can you instead attach a patch formatted using the instructions at https://clojurescript.org/community/patches

Answer 3

Comment made by: namenu

Sorry to bother you.

Here's a new patch.

Answer 4

Comment made by: mfikes

CLJS-2959.patch LGTM, passes CI and Canary.

Answer 5

Comment made by: thegeez

CLJS-2959-2.patch updates CLJS-2959.patch to name the test case after the CLJS issue number, instead of the CLJ issue number.

Answer 6

Reference: https://clojure.atlassian.net/browse/CLJS-2959 (reported by mfikes)