matsim-vsp / freight-dfg17

GNU General Public License v2.0
0 stars 1 forks source link

Merge changes #21

Closed JWJoubert closed 4 years ago

JWJoubert commented 4 years ago

Some code fixes in receiver after merging most recent code refactorings.

kainagel commented 4 years ago

The build is failing because LFS is above its quota. @kt86 is working on moving this repo out of LFS. I don't really want to pay more, so we either need to wait until the monthly quota is refilled, or until the repo is moved out of LFS.

kt86 commented 4 years ago

removing git lfs is prepared in #22. Waiting for the "go" of @JWJoubert regarding the fhandling of the files, that were stored in lfs.

kt86 commented 4 years ago

Look like there is a hickup between github and travis: Travis did not noticed any changes here (commits 4caaa51 and 45ec8a5). I will close this PR and open a new one.

kt86 commented 4 years ago

It did not worked with the other PR #22 as well. Found out that travis cannot authorize to github. This seems in most cases to be related to the fact, that one ran out of the (100) free builds on travis for private repos : https://docs.travis-ci.com/user/common-build-problems/

kt86 commented 4 years ago

All tests are passing locally. --> merging should be fine