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

Uberjar does not merge files (except for data_readers.clj) #28

Open
hlship opened this issue Jul 25, 2023 · 3 comments
Open

Uberjar does not merge files (except for data_readers.clj) #28

hlship opened this issue Jul 25, 2023 · 3 comments

Comments

@hlship
Copy link
Contributor

hlship commented Jul 25, 2023

Expected Behavior

When multiple libraries contain the same file, they should be merged together. Specifically, .edn files should be deep-merged together.

Actual Behavior

Last file read overwrites earlier file written.

@puredanger
Copy link

Seems like a surprising expectation to me. Really, there are a range of behaviors you might want (log4j config files are notoriously weird for example). tools.build abstracts this into an api and provides a variety of available handlers.

@hlship
Copy link
Contributor Author

hlship commented Jan 3, 2024

The solution is flexible in how such a merge occurs.

@hlship
Copy link
Contributor Author

hlship commented Jan 3, 2024

#29

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

2 participants