Looks good to me (though includes some minor whitespace changes). But I can't merge it. On Tue, Jan 16, 2018 at 4:41 PM, Amit Saha <amitsaha.in(a)gmail.com&gt; wrote: > Hi all, > > Can I please get a review of > https://pagure.io/fedora-kickstarts/pull-request/326 as a "fix" for > https://bugzilla.redhat.com/show_bug.cgi?id=1506888 ? > > Thanks, > Amit. > > > _______________________________________________ > spins mailing list -- spins(a)lists.fedoraproject.org > To unsubscribe send an email to spins-leave(a)lists.fedoraproject.org > > _______________________________________________ spins mailing list -- spins(a)lists.fedoraproject.org To unsubscribe send an email to spins-leave(a)lists.fedoraproject.org

On Thu, 18 Jan 2018 at 8:25 am, Amit Saha <amitsaha.in(a)gmail.com&gt; wrote: > Thanks Rex. > > Can someone who can merge please take a look and push it? Gentle reminder. Can someone please look at the PR?

> > > On Thu, 18 Jan 2018 at 3:58 am, Rex Dieter <rdieter(a)gmail.com&gt; wrote: > >> Looks good to me (though includes some minor whitespace changes). >> >> But I can't merge it. >> >> On Tue, Jan 16, 2018 at 4:41 PM, Amit Saha <amitsaha.in(a)gmail.com&gt; >> wrote: >> >>> Hi all, >>> >>> Can I please get a review of >>> https://pagure.io/fedora-kickstarts/pull-request/326 as a "fix" for >>> https://bugzilla.redhat.com/show_bug.cgi?id=1506888 ? >>> >>> Thanks, >>> Amit. >>> >>> >>> _______________________________________________ >>> spins mailing list -- spins(a)lists.fedoraproject.org >>> To unsubscribe send an email to spins-leave(a)lists.fedoraproject.org >>> >>> >> _______________________________________________ >> spins mailing list -- spins(a)lists.fedoraproject.org >> To unsubscribe send an email to spins-leave(a)lists.fedoraproject.org >> >