search

healthfinch / depstar

A clj-based uberjarrer. See version maintained by Sean for up to date releases.
https://github.com/seancorfield/depstar
104 stars 24 forks source link

Supporting :merge-edn clash strategy for other files #11

Open weavejester opened 5 years ago

weavejester commented 5 years ago

Would you accept a PR for adding an option to extend the :merge-edn clash strategy to other files? For example, adding a --merge-edn option with a comma-separated list of resources. Duct relies on a file called duct_hierarchy.edn that words similarly to data_readers.edn and needs to be merged when compiled.

seancorfield commented 5 years ago

Feel free to open an issue against my (well-maintained) fork https://github.com/seancorfield/depstar

seancorfield commented 4 years ago

@weavejester I don't recall seeing an issue or PR against my fork for this -- is this problem still something you need to be solved?

weavejester commented 4 years ago

@seancorfield It is, at least once I try out tools.deps again, as I've been working on Leiningen-based projects recently. I think Badigeon has this functionality now, so potentially I could use that instead of depstar.

seancorfield commented 4 years ago

OK. Happy to entertain an enhancement issue if you do come back to depstar at some point! Maybe this issue should be closed out (since I don't think Healthfinch intends to enhance their depstar at this point)?