Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [paho-dev] Submiting PR to Paho-C

Np, 
Also if you change the commit SHA-1, you can ask GH to close the PR by
adding a "close #2" in your commit message.

On Mon, Oct 26, 2015 at 07:07:29AM -0700, Ian Craggs wrote:
> Thanks Julien, I'll try that. :-)
> 
> On 10/23/2015 02:44 PM, Julien Vermillard wrote:
> > In my understanding "mirroring" is a way of "hosting" :)
> > If the CLA and sign-off is fine, I think the best is on your local clone to do something like:
> >
> > git checkout -b the-branch-for-the-crontrib
> > git pull https://github.com/aribeironovaes/paho.mqtt.c.git master
> >
> > git checkout master
> > git merge (--no-ff or --ff-only depending) the-branch-for-the-crontrib
> > git push origin master
> >
> > Then if you don't rewrite the commit ID github will close the PR.
> >
> > On Fri, Oct 23, 2015 at 06:01:02AM -0700, Ian Craggs wrote:
> >> Yes, but this page:
> >>
> >> https://wiki.eclipse.org/Development_Resources/Contributing_via_Git#via_GitHub
> >>
> >> on the Eclipse wiki says:
> >>
> >> If the project repository is hosted on GitHub, you can issue a standard GitHub
> >> pull request. Ensure that you have a CLA and that the commit record is properly
> >> "Signed-off-by" before you issue the request.
> >>
> >> and the Paho repositories are mirrored on GitHub, so I thought I should learn
> >> :-)  "Mirroring" may not be the same as "hosting", but I think the fact that
> >> the repositories are accessible from GitHub does set some expectations.  Angelo
> >> has signed the CLA and signed the contribution off.
> >>
> >> Ian
> >>
> >> On 10/23/2015 12:56 PM, Ian Skerrett wrote:
> >>
> >>      Would it be easier if Angelo just submits his pull request via Gerrit?
> >>
> >>
> >>      Sent from Samsung Mobile
> >>
> >>
> >>      -------- Original message --------
> >>      From: Ian Craggs
> >>      Date:10-23-2015 06:24 (GMT-05:00)
> >>      To: paho-dev@xxxxxxxxxxx
> >>      Subject: Re: [paho-dev] Submiting PR to Paho-C
> >>
> >>      Hi Angelo,
> >>
> >>      thanks for the contribution.  I'm not used to accepting pull requests from
> >>      Github (usually bugzilla or Gerrit) so I'm going to have to work out how to
> >>      handle it.    I hope to get to it over the weekend.
> >>
> >>      Ian
> >>
> >>
> >>      On 10/22/2015 08:54 PM, Angelo Ribeiro wrote:
> >>
> >>          Hi,
> >>
> >>          I would like to know if I'm following the right process to contribute
> >>          back to paho-c.
> >>
> >>          I have signed the CLA and have submit a pr on github.
> >>
> >>          Here is the PR to add 64 bits (Windows) configuratio on our project
> >>          files...
> >>
> >>          https://github.com/eclipse/paho.mqtt.c/pull/2
> >>
> >>          Do I have to follow any other process or use other tools to have this
> >>          accepted?
> >>
> >>          I have submit this on October 12th and not answer yet.
> >>
> >>          I have another PR (A fix this time), but I would like to check if I'm
> >>          following the right process.
> >>
> >>          Thanks,
> >>          ---
> >>          Angelo Ribeiro
> >>          Msc - Federal University of Pernambuco
> >>          Team in Training Alumni Triple Crowned! :)
> >>
> >>         
> >>
> >>          _______________________________________________
> >>          paho-dev mailing list
> >>          paho-dev@xxxxxxxxxxx
> >>          To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> >>          https://dev.eclipse.org/mailman/listinfo/paho-dev
> >>
> >>
> >>      --
> >>      Ian Craggs
> >>      icraggs@xxxxxxxxxx                 IBM United Kingdom
> >>      Paho Project Lead; Committer on Mosquitto
> >>
> >>
> >>
> >>     
> >>      _______________________________________________
> >>      paho-dev mailing list
> >>      paho-dev@xxxxxxxxxxx
> >>      To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> >>      https://dev.eclipse.org/mailman/listinfo/paho-dev
> >>
> >>
> >> --
> >> Ian Craggs
> >> icraggs@xxxxxxxxxx                 IBM United Kingdom
> >> Paho Project Lead; Committer on Mosquitto
> >>
> >>
> >> _______________________________________________
> >> paho-dev mailing list
> >> paho-dev@xxxxxxxxxxx
> >> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> >> https://dev.eclipse.org/mailman/listinfo/paho-dev
> >
> 
> -- 
> Ian Craggs
> icraggs@xxxxxxxxxx                 IBM United Kingdom
> Paho Project Lead; Committer on Mosquitto
> 
> _______________________________________________
> paho-dev mailing list
> paho-dev@xxxxxxxxxxx
> To change your delivery options, retrieve your password, or unsubscribe from this list, visit
> https://dev.eclipse.org/mailman/listinfo/paho-dev

-- 
Julien Vermillard
Mobile: +33680328665
jvermillard@xxxxxxxxxxxxxxxxxx :: https://www.sierrawireless.com


Back to the top