Dive plan: Taking into account isobaric counterdiffusion

Stefan Fuchs sfuchs at gmx.de
Thu Jan 4 00:24:40 PST 2018


Hello Willem,

Am 04.01.2018 um 07:13 schrieb Willem Ferguson:
>
>> Can you do a Github PR for this? For me it's much easier to discuss
>> it there and also to pull and test it :-)
>>
> Done. The title of the PR is a bit obscure, not intended.
> Kind regards,
> willem
Honestly the whole PR looks a little bit "obscure" or in other words -
there went s.th. wrong ;-)

Maybe the way you did the PR (file upload?!) is not the best one. But
even if it works there are two other mistakes beside the PR title:
- The file has to go in /core/plannernots.c
- The commit title is also wrong

Maybe better to do the PR in the way of pushing the change to your own
GitHub repo and then do the PR from there.
I'm also no real pro but my flow usually looks like:
- Change is available in my local git repo in a branch I call "mystable"
including 1-n commits
- Local branch master is up to date
- git checkout -b feature_whatever
- git rebase -i master
  Remove everything which is NOT part of the change
- git push https://github.com/sfuchs79/subsurface.git feature_whatever
- Then do the PR on GitHub

Best regards
Stefan

-- 

Stefan Fuchs
E-Mail: sfuchs at gmx.de <mailto:sfuchs at gmx.de>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20180104/33f2fc42/attachment.html>


More information about the subsurface mailing list