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.

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

many minor adjustments for ClojureScript #357

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

timothypratley
Copy link

This patch doesn't solve everything.
It does get significantly further in the compilation (builds with warnings),
but more work would be required to make it actually work correctly.
Suggestions/collaboration sought on achieving full compatibility.

This patch doesn't solve everything.
It does get significantly further in compilation (builds with warnings),
but more work would be required to make it actually work correctly.
Suggestions/collaboration sought on achieving fully compatible.
@kordano
Copy link
Member

kordano commented Jul 8, 2021

Thanks for your contributions @timothypratley. Currently we're adjusting konserve for sync and async versions as preparations for the cljs port. Maybe @whilo and @groundedsage could have a look.

@timothypratley
Copy link
Author

You're welcome! Thanks for the heads up about Konserve.

@whilo
Copy link
Member

whilo commented Jul 9, 2021

@timothypratley Thanks for your work! The rebase of the cljs support is planned after we have landed the current changes for scaling out in our 1.0 release. I assume you are aware of the cljs-support branch. We will not do much more work on it, but reapply the changes there in a slightly different way to the current development branch, so your work here is definitely valuable. We are very happy to discuss collaborating on it if you are interested (best on Discord).

@whilo whilo mentioned this pull request Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

3 participants