:+1:

--
Thiago Belem
HE:labs developer
On Jan 31, 2014 10:37 PM, "Sylvestre Mergulhão" <notifications@github.com>
wrote:

> In lib/jumpup/heroku/integrate.rb:
>
> > @@ -70,18 +70,15 @@ def add_remote
> >        def check
> >          puts "--> Checking if there's already someone integrating to #{app}"
> >
> > -        var = run_with_clean_env("heroku config -s --app #{app} | grep INTEGRATING_BY", true).strip
> > -        integrating_by = var.split('=')[1]
> > -
> > -        integrating_by.strip! unless integrating_by.blank?
> > -
> > -        if integrating_by != user and not integrating_by.blank?
> > +        if integrating_by and integrating_by != user
>
> @TiuTalk <https://github.com/TiuTalk> may we refactor this to something
> like if not_integrating_by?(user) ?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/Helabs/jumpup-heroku/pull/20/files#r9362891>
> .
>