+1 to consistency, change to single make on the factory itself to
On Aug 21, 2013 5:44 PM, "Walden Raines" <notifications@github.com> wrote:

> In engines/bastion/app/assets/bastion/bastion.js:
>
> > @@ -58,6 +58,8 @@
> >  //= require "bastion/utils/utils.module"
> >  //= require "bastion/utils/routes.factory"
> >
> > +//= require "bastion/system-groups/system-groups.module.js"
> > +//= require "bastion/system-groups/system-groups.factory.js"
>
> Yeah, I went back and forth on that actually.
>
> Keep in mind that this already passed one round of reviews to get into
> master (#2369 <https://github.com/Katello/katello/issues/2369>), I just
> moved it.
>
> I can rename if it really bothers you.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Katello/katello/pull/2786/files#r5910264>
> .
>