Re: Freeze break request? Pagure update (round 2)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 05, 2015 at 11:17:46AM +0200, Pierre-Yves Chibon wrote:
> On Wed, Aug 05, 2015 at 10:54:45AM +0200, Pierre-Yves Chibon wrote:
> > On Mon, Aug 03, 2015 at 04:06:15PM -0600, Kevin Fenzi wrote:
> > > On Mon, 3 Aug 2015 22:45:33 +0200
> > > Pierre-Yves Chibon <pingou@xxxxxxxxxxxx> wrote:
> > > 
> > > > Hi everyone,
> > > > 
> > > > So there has been some changes to pagure that I would like to push
> > > > forward, here is the detail:
> > > > 
> > > > - Make the SSH protocol explicit for the SSH URLs (Till Maas)
> > > > - Improve/Fix/Rework pagure's documentation
> > > > - Rework the doc server to make it display what's in the repo
> > > > directly (so you can directly put there the output of your sphinx
> > > > build, or your rst/md/text files)
> > > > - Install properly the post-receive hook on the tickets and requests
> > > > hooks (hotfixed in prod)
> > > > - Give proper attribution to the pagure logo to Micah Deen in the
> > > > documentation
> > > > 
> > > > As you see, nothing major for the main application, but the change to
> > > > the doc server and doc integration into the main app would be helpful.
> > > > 
> > > > Thoughts on pushing this forward?
> > > 
> > > Sounds ok to me... but is there already a release/tag so we can look at
> > > the changes in git?
> > > 
> > > +1 in any case, but would be nice to look over the exact changes in
> > > case there's a issue someone notes. 
> > 
> > I have added another few bug fixes:
> > - Increase pull request text field lengths to 80 (Till Maas)
> > - Let anyone open remote pull-request against a project (not just the admin of
> >   the project to which we want to open the PR)
> > - Check that the parent (target) accepts PRs instead of checking the current
> >   repo
> > - Fix the branch selection to avoid blank entries
> > 
> > The entire diff is at: http://fpaste.org/251663/64745143/
>  
> I'd like to add this fix as well:
> https://pagure.io/pagure/pull-request/322 (if someone is kind enough to review
> it)
> 
> It's fixing a bug for which I'm receiving emails notifications.
> 
> Thanks,
> Pierre

+1

Attachment: signature.asc
Description: PGP signature

_______________________________________________
infrastructure mailing list
infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/infrastructure

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux