Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Can we get this to work on cljs? #43

Closed
shlomi-nx opened this issue Jan 6, 2019 · 7 comments
Closed

Can we get this to work on cljs? #43

shlomi-nx opened this issue Jan 6, 2019 · 7 comments

Comments

@shlomi-nx
Copy link

Thanks!
@cloojure
Copy link

Yes, I second the request!
Thanks.

@borkdude
Copy link
Contributor

borkdude commented Jan 26, 2021 •
edited

I think if someone offered a PR making this library .cljc, it would get accepted. What do you think @slipset and @danielcompton?

@slipset
Copy link
Member

slipset commented Jan 26, 2021

Deffo interesting. Maybe so interesting that I'll try to do one myself.

@plexus plexus mentioned this issue Aug 24, 2021
Merged
@plexus
Copy link
Contributor

plexus commented Aug 24, 2021

See #59 for an implementation of ordered-map.

@slipset slipset reopened this Sep 21, 2021
@vemv
Copy link

vemv commented Oct 11, 2021

Good to close?

@filipesilva
Copy link

ordered-map is implemented but ordered-set is not. Not sure if anyone has a lot of interest in the latter though.

@borkdude
Copy link
Contributor

Perhaps close this and make another issue for ordered-set + CLJS, to keep things scoped.

@slipset slipset mentioned this issue Oct 11, 2021
Open
@slipset slipset closed this as completed Oct 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@plexus @borkdude @vemv @filipesilva @slipset @cloojure @shlomi-nx