Configuration instance builder

Lukas Krejci lkrejci at redhat.com
Mon Jul 15 09:30:17 UTC 2013


I forgot about this completely :)

Pushed.

On Sunday, July 14, 2013 07:28:59 John Sanda wrote:
> I don't see this in master yet. Are you still planning to merge it? Let me
> know when you do. I'd like to use it.
> On May 22, 2013, at 1:34 PM, Lukas Krejci <lkrejci at redhat.com> wrote:
> > Hi all,
> > 
> > on a tangent to my current work on the provisioning subsystem I created a
> > fluent builder for Configuration instances.
> > 
> > In my opinion, this will greatly simplify the task of creating the complex
> > configuration objects, mainly while writing the tests.
> > 
> > I placed the builder directly in the Configuration class so that it is
> > more easily discoverable, so this has an impact on our remote API,
> > because we'll start exposing this class.
> > 
> > I don't see anything wrong with it though, it should help people in the
> > CLI, too.
> > 
> > The code along with tests (that cover the builder code by 100%) is in this
> > commit:
> > 
> > https://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?h=lkrejci/configurat
> > ion-builder&id=20beed909fc2a647434244bd5bcb00839a683606
> > 
> > If you have any ideas for improvements of the API, its usage or are
> > opposed to this going to master, please speak up.
> > 
> > I am going to merge this into master in a couple of days.
> > 
> > Thanks,
> > 
> > Lukas
> > _______________________________________________
> > rhq-devel mailing list
> > rhq-devel at lists.fedorahosted.org
> > https://lists.fedorahosted.org/mailman/listinfo/rhq-devel
> 
> _______________________________________________
> rhq-devel mailing list
> rhq-devel at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/rhq-devel


More information about the rhq-devel mailing list