On Fri, Dec 21, 2012 at 9:35 AM, Pascal Obry <[email protected]> wrote: > A new version of this branch is ready. See pull request.
thx will try to look at it soonish, but if i'm not reactive feel free to ask on IRC for another reviewer > > BTW, what the best procedure to discuss enhancements? > > 1. Should this be discussed here before a pull-request? > if you think your idea might not be to the dev's liking and is a substential effort it's better to discuss it here first if it's trivial and/or non-controversial, you can code right away > 2. Should I pull-request and discuss vis comments on GitHub? > we tend to comment pull requests directly on the pull request, it's more handy to follow the conversation so yeah, discuss the "grand scheme of things" here, once the code is ready for merge do a pull request and we start discussing the specific of the code in the pull request Cheers Boucman ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ darktable-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/darktable-devel
