-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Distribution files? #142
Comments
Hey @Frozenlock thanks for raising this. We investigated this in the past (#81) and it's something we'd be happy to add at some point. Right now we are focused on priorities from our immediate roadmap so we won't be able to address this right away, but we'd be open to PRs from anyone from the community who would like to work on this :) |
I'd like to contribute to solve this issue. However, I am not entirely sure how to approach it. At first I thought to remove |
Great to hear @vinifala!! I am not familiar with the particulars of what is needed for Clojurescript. The ideal would be if we also built a single bundle for distribution. Eg in |
Please take a look at the prior work here: #81 |
Closed by #186 |
@Frozenlock let me know how you go getting it onto https://cljsjs.github.io/ . I can add a link from this repo when it is up. Cheers |
@Frozenlock any update on this? |
@alexreardon Sorry for the delay, I'll try to do it in the next few days. |
All good! |
Done! Thank you so much for the distribution files! |
So great @Frozenlock !!! I'll add a link to the readme! |
Are there any examples of usage of the library in cljs? (would be super helpful) |
In the Clojurescript world we often package JS libraries for ease of use.
When doing so, it's encouraged to use the official distribution files (
dist/
). This helps us upgrade packages and keep them in sync with the JS libraries.I was ready to package and use this project but couldn't find those files 😢
Would it be possible to add this into react-beautiful-dnd?
The text was updated successfully, but these errors were encountered: