Skip to content

Conversation

daveliepmann
Copy link
Contributor

Closes #360.

Because merge has a large surface area of somewhat-forgiving conj-driven undefined behavior, I chose not to test e.g. (merge '(1 2 3) 1) very closely. See last testing form; open to discussion.

Copy link
Member

@jeaye jeaye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Dave. Just one suggested addition.

@daveliepmann daveliepmann requested a review from jeaye September 9, 2025 06:03
@jeaye jeaye merged commit b908bb1 into jank-lang:main Sep 9, 2025
2 checks passed
@jeaye
Copy link
Member

jeaye commented Sep 9, 2025

Nice work, Dave!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

clojure.core/merge
2 participants