Skip to content
This repository has been archived by the owner on Apr 27, 2020. It is now read-only.

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

Open
weavejester opened this issue Dec 3, 2018 · 4 comments
Open

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

weavejester opened this issue Dec 3, 2018 · 4 comments

Comments

@weavejester
Copy link

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
Copy link
Contributor

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

@seancorfield
Copy link
Contributor

@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
Copy link
Author

@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
Copy link
Contributor

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants