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

Merge data_readers.clj on the fly #1

Closed
tonsky opened this issue Apr 29, 2019 · 5 comments
Closed

Merge data_readers.clj on the fly #1

tonsky opened this issue Apr 29, 2019 · 5 comments

Comments

@tonsky
Copy link
Owner

tonsky commented Apr 29, 2019

No description provided.

@thenonameguy
Copy link

thenonameguy commented Jul 9, 2019

It would be nice if merging of arbitrary classpath resources was supported by the lib. The generality could pave the way for supporting Duct-style applications:
duct-framework/duct#92

Essentially the :uberjar-merge-with option of leiningen should be extracted:
https://github.com/technomancy/leiningen/blob/87a6af9a12d5c479bfbb51f50b85c3fa4c2650a0/leiningen-core/src/leiningen/core/project.clj#L242

@tonsky
Copy link
Owner Author

tonsky commented Jul 9, 2019

Thanks for the pointer!

@WhittlesJr
Copy link

WhittlesJr commented Aug 13, 2020

I ran into this as well. I need to be able to merge resource files a la :uberjar-merge-with.

@tonsky
Copy link
Owner Author

tonsky commented Aug 13, 2020

@WhittlesJr just to be clear: you need to merge data_readers or you need a custom merger?

I plan to fix these in the nearest future

@WhittlesJr
Copy link

I believe what I need is a custom merger. I'm working with duct just like @thenonameguy, and I need to merge a bunch of .edn files that shows up with the same classpath from multiple projects. Here's the issue as experienced by a boot user: duct-framework/duct#102

@tonsky tonsky closed this as completed in c6af8dd Aug 19, 2020
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

No branches or pull requests

3 participants