SSPkrolik / nimongo

Pure Nim lang MongoDB driver
http://sspkrolik.github.io/nimongo
MIT License
101 stars 20 forks source link

several fixes #57

Closed timotheecour closed 5 years ago

timotheecour commented 5 years ago

/cc @yglukhov @SSPkrolik

timotheecour commented 5 years ago

hmm, indeed, nim c -r nimongo/bsontest is mentioned in .travis.yml and doesn't fail... sounds like a pre-existing bug

timotheecour commented 5 years ago

@SSPkrolik @yglukhov I tested locally (based on output from https://travis-ci.com/timotheecour/nimongo/jobs/159187423) , it turns out travis.yml is buggy, can we please merge this and figure out travis after?

mkdir /tmp/d02 && cd /tmp/d02
git clone --depth=50 --branch=master https://github.com/timotheecour/nimongo.git timotheecour/nimongo
docker pull yglukhov/nim-base

# passes
docker run -v "$(pwd):/project" -w /project yglukhov/nim-base run "nimble install -y && nim c -r nimongo/bson && nim c -r nimongo/bsontest"

# also passes, with random string
docker run -v "$(pwd):/project" -w /project yglukhov/nim-base run "nimble install -y && nim c -r nimongo/bson && nim c -r adfasfjalfjaslfjasdlfjalsjfalsjfasljfalsdfj"
timotheecour commented 5 years ago

@SSPkrolik @yglukhov PTAL, I fixed travis, there was a pre-existing bug (run vs bash); also I adapted travis to run nimble test_ci so we can test both nimble test (locally, if there's a mongod server) vs nimble test_ci

in a future PR, we should make travis run nimble test and add dependency on mongod in travis => https://github.com/SSPkrolik/nimongo/issues/62

timotheecour commented 5 years ago

/cc @yglukhov I added some commits ; lemme know if u want these in another PR

timotheecour commented 5 years ago

ping

SSPkrolik commented 5 years ago

@timotheecour thanks a lot for the contribution!