On (01/09/16 10:30), Jakub Hrozek wrote: >On Thu, Sep 01, 2016 at 09:49:50AM +0200, Petr Cech wrote: >> On 08/31/2016 10:28 AM, Jakub Hrozek wrote: >> > Hi, >> > >> > I documented workflow that we could use for submitting PRs: >> > https://fedorahosted.org/sssd/wiki/GithubWorkflow >> > >> > It's quite similar to what the FreeIPA team uses (although I don't think >> > they publicly document it yet). >> > >> > Comments or edits welcome. If there are none, I'll link the page from >> > the Contribute page later. >> >> Hello, >> >> I have note to 'Submitting a pull-request'. >> Now we advice to: >> * use github repo, >> * fork it, >> * add it like remote. >> >> If I understand correctly there is no reason >> for contributors to clone original >> fedorahosted repo. Is it right? > >Yeah, you can clone github/sssd/sssd.git directly. Only those who want >to push need to clone the fedorahosted repo. > >(Cloning fedorahosted would also make sure you always get the canonical >sources i ncase the fedorahosted->github mirroring breaks or in case we >move away from github in the future..) > >> >> We should rewrite Contribute wiki page in this manner. > >Yes, I'll do that later today I guess. I do not agree. github workflow wiki says: "Github pull requests are the preferred way" It does not say it is a only way how to contribute So fedorahosted repo is a primary repo therefore it should be listed in the "Contribute wiki" page.
BTW editing .git/config is error prone; especially if name of remote repo is not a "github" e.g. hint [remote "github"] url = https://github.com/SSSD/sssd.git fetch = +refs/heads/*:refs/remotes/github/* fetch = +refs/pull/*/head:refs/remotes/github/pull/* It would be better to use "git config" e.g GITHUB_REMOTE="gh" git config --add remote.$GITHUB_REMOTE.fetch \ "+refs/pull/*/head:refs/remotes/$GITHUB_REMOTE/pull/*" LS _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org