Allow omitting Dispatch client code #322

Closed
wants to merge 2 commits into
from

Projects

None yet

2 participants

@liff
Contributor
liff commented May 6, 2015

Adds a flag and relevant settings for disabling generation of Dispatch client code.

Sometimes we may want to use our own HTTP client without bringing in an unused Dispatch dependency.

Olli Helenius added some commits May 6, 2015
Olli Helenius Change Config to a regular class
This allows us to add more fields to it. At the moment adding just one takes it over the 22 field limit.
ad34805
Olli Helenius Add parameter for skipping the generation of Dispatch client code
Adds:
  --no-dispatch-client – command line parameter
  enableDispatchClient – SBT setting
  scalaxb.enableDispatchClient – Maven setting

The default value is ‘true’ so that current behavior is not changed.
4ba9200
@eed3si9n
Owner

Sorry I haven't commented on this PR yet.
I was thinking maybe having Seq of sealed trait is a better way of representing config. I have to experiment and see how it would look.

@eed3si9n eed3si9n added a commit that referenced this pull request Jul 20, 2015
@eed3si9n Olli Helenius + Manually merge #322 63377f1
@eed3si9n
Owner

Closing this in favor of #345

@eed3si9n eed3si9n closed this Jul 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment