Merge should work for projects own resources too

[kul]

Here is the problem i am facing which i think would be pretty common for others too.
One of the data_readers.clj is inside one of the jar files and other is in my resources folder.
Since resources gets copied in a different step from uber step, the .exists method in unpack-jar will return false for it causing the uber step to overwrite the fileset without actually merging.

I guess the mergers would need to be applied at commit! too. WDYT?

[micha]

Nice! Thanks for reporting this! It sounds totally reasonable to me. I'll work up a patch and run it by @tobias for review.

Are you also recommending that we merge data_readers.clj files? What about deps.cljs? Are there any other non-namespaced Clojure/ClojureScript files in JARS that we need to merge when uberizing? A separate ticket for those would be appropriate, probably.

[micha]

Whoops, my bad, data_readers.clj is already in the merge list, please disregard that part 😄

[kul]

Works Perfectly! Thanks @micha

[tobias]

Thank you @kul for catching this, and thank you @micha for fixing it.