depstar icon indicating copy to clipboard operation
depstar copied to clipboard

Supporting :merge-edn clash strategy for other files

Open weavejester opened this issue 6 years ago • 4 comments

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.

weavejester avatar Dec 03 '18 10:12 weavejester

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

seancorfield avatar Aug 06 '19 02:08 seancorfield

@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?

seancorfield avatar Apr 22 '20 01:04 seancorfield

@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.

weavejester avatar Apr 22 '20 02:04 weavejester

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)?

seancorfield avatar Apr 22 '20 02:04 seancorfield