Re: [Avogadro-devel] Switching to GitHub pull requests for review/merge?

2014-10-08 Thread Marcus D. Hanwell 
On Mon, Oct 6, 2014 at 10:34 PM, Marcus D. Hanwell
marcus.hanw...@kitware.com wrote:
 Hi,

 Would there be any objections to switching to GitHub pull requests for
 development? The Kitware instance of Gerrit will be upgraded at some
 point, removing the topic branch functionality. Gerrit feels a little
 heavy/clunky, and I feel like GitHub has come a long way since we made
 the switch to Gerrit.

Geoff gave me a +10 off-list, I will work on clearing the Gerrit
backlog, planning to move to GitHub pull requests unless I hear
objections. We are looking at a project called GitLab for several
Kitware projects that are considering moving from Gerrit (they are
also considering GitHub). This may be interesting, but it effectively
mirrors much of the functionality of GitHub.

Thanks,

Marcus

--
Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer
Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports
Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper
Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer
http://pubads.g.doubleclick.net/gampad/clk?id=154622311iu=/4140/ostg.clktrk
___
Avogadro-devel mailing list
Avogadro-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/avogadro-devel

Re: [Avogadro-devel] Switching to GitHub pull requests for review/merge?

2014-10-08 Thread Geoffrey Hutchison 
 Geoff gave me a +10 off-list, I will work on clearing the Gerrit

Sorry, I thought that was on-list. Here's the message:

 We've been using GitHub pull requests for Open Babel and I lurk on a few 
 other projects (i.e., cclib) where it works really well.
 
 I think a big advantage is the public visibility on GitHub, both of 
 networks (i.e., contributors) and the pull requests themselves.
 
 There's also Travis CI integration for pull requests, so if tests are set up 
 correctly, you can see if there are issues from Travis.
 
 -Geoff

--
Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer
Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports
Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper
Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer
http://pubads.g.doubleclick.net/gampad/clk?id=154622311iu=/4140/ostg.clktrk
___
Avogadro-devel mailing list
Avogadro-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/avogadro-devel