https://help.github.com/articles/checking-out-pull-requests-locally/
Script a bot to scrape the patches and post to phabricator? On Fri, 1 Feb 2019 at 04:22, Roman Lebedev via llvm-dev < llvm-...@lists.llvm.org> wrote: > On Thu, Jan 31, 2019 at 8:29 PM David Greene via cfe-dev > <cfe-...@lists.llvm.org> wrote: > > > > Mehdi AMINI <joker....@gmail.com> writes: > > > > > What is the practical plan to enforce the lack of merges? When we > > > looked into this GitHub would not support this unless also forcing > > > every change to go through a pull request (i.e. no pre-receive hooks > > > on direct push to master were possible). Did this change? Are we > > > hoping to get support from GitHub on this? > > > > > > We may write this rule in the developer guide, but I fear it'll be > > > hard to enforce in practice. > > > > Again, changes aren't going through git yet, right? Not until SVN is > > decommissioned late this year (or early next). SVN requires a strict > > linear history so it handles the enforcement for now. > > > My personal opinion is that when SVN is decomissioned we should use pull > > requests, simply because that's what's familiar to the very large > > development community outside LLVM. It will lower the bar to entry for > > new contributors. This has all sorts of implications we need to discuss > > of course, but we have some time to do that. > My personal opinion is an opposite of that one. > > *Does* LLVM want to switch from phabricator to github pr's? > I personally don't recall previous discussions. > Personally, i hope not, i hope phabricator should/will stay. > > Low bar for entry is good, but not at the cost of raising the bar > for the existing development community. > In particular, i'm saying that github's ui/workflow/feature set > is inferior to that one of phabricator. > > Is the low entry bar the only benefit? > While it is good, it should not be the only factor. > The contributors will already need to know how to build LLVM, > write tests, make meaningful changes to the code. > Additionally having to know how to work with phabricator > isn't that much to ask for, considering the other prerequisites.. > > > If we don't use PRs, then we're pretty much on the honor system to > > disallow merges AFAIK. > > > > -David > Roman. > > > _______________________________________________ > > cfe-dev mailing list > > cfe-...@lists.llvm.org > > https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev > _______________________________________________ > LLVM Developers mailing list > llvm-...@lists.llvm.org > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >
_______________________________________________ lldb-dev mailing list lldb-dev@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev