Add combiner-fn option #6

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

Add combiner-fn to support things like having multiple values for the same switch, e.g.:

-a 1 --config val1 --config val2

An example is in the tests.

Contributor

gar3thjon3s commented May 9, 2012

Hi Nathan,

Clojure does not accept pull requests on github -- you need to create a ticket in JIRA and attach a patch to the ticket. You also need to have signed a contributor agreement before you can do that:

http://dev.clojure.org/jira
http://clojure.org/contributing

As mentioned by jszakmeister, there is another patch similar to this that I was waiting on the author to sign and get his CA before applying. Sounds like he prefers your solution and it looks good to me, so if you already have your CA go ahead and create the ticket with the patch and I will apply and release.

Cheers

gar3thjon3s closed this Aug 9, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment