Closed tschoonj closed 7 years ago
@reinh-bader I fixed the Travis-CI builds finally.
Also, I saw that you forgot to add the new nlfit.finc files to Makefile.am, which I took care of.
OK, thanks.
If I may suggest a different workflow for non-trivial additions:
git checkout -b my-new-branch
git push -u origin my-new-branch
Generally speaking, committing directly onto master is discouraged nowadays as it can be quite dangerous (git push --force
!). In fact in the settings of the repository, you can even disable committing to master altogether by making it a so-called protected branch.
Rather complicated. I'm game, though, except that I don't fully understand the following point: The pull request goes to the branch, but in step 7. it is merged into the master - correct? Does the web interface support doing this?
Cheers
Absolutely, this is in fact what you did yesterday when you closed the PR by merging it in.
There are three ways of doing this (click in a future PR on the merge button to get the dropdown list):
OK, thanks. The next commit will still go to master (I've already done some changes), but I'll start using the new workflow after that. I'll also document that workflow in README_WORKFLOW, because after a long interval of inactivity I'm bound to forget about it :-/
Actually, assuming you haven't made any commits locally on master that you haven't pushed yet, you will first need to pull in my latest commit with the Travis-CI changes locally with a simple git pull
.
Otherwise you will need to rebase your local master against the remote master (check first that you are on master with git status, just in case 😄 ):
git fetch
git rebase origin/master
git push
Also, you can make a new branch whenever you want, even when there are already modified files.
Another advantage of not working on master directly is that it frees you from having to push your changes immediately to github, as it is always possible that someone has pushed commits to master already!
Hmm, the branches CI tests seem to have problems with the MAC builds, However, it seems that the build doesn't start up properly. Linux builds are fine.
Well I managed to get them right in the end 😄
Actually the main problem was that the new nlfit.finc files had not been added to Makefile.am, which caused make distcheck
to fail. This took me quite some time to figure out 😄
The trouble is that I can't see any messages on the console. If I try to open it, I only get the message "Hang tight, the log cannot be shown until the build has started." I will try to check "make distcheck" on a separate system tomorrow (on the devel system, there is some trouble with tar due to my high GID on that system).
You mean the Mac OS X builds right? See my latest comment in #15
Please do not merge until the Travis-CI build is green