Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated travis.yml file for better test coverage #69

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

camiloforero
Copy link

Seeing how Travisl will run with this configuration

@camiloforero
Copy link
Author

Ok, this failed because of when all of Github failed, trying to get the tests to run again

@camiloforero camiloforero reopened this Oct 22, 2018
@mongoid-bot
Copy link

mongoid-bot commented Oct 22, 2018

1 Warning
⚠️ Unless you’re refactoring existing code, please update CHANGELOG.md.

Here's an example of a CHANGELOG.md entry:

* [#69](https://github.com/mongoid/mongoid-geospatial/pull/69): Updated travis.yml file for better test coverage - [@camiloforero](https://github.com/camiloforero).

Generated by 🚫 danger

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.724% when pulling 52446aa on camiloforero:master into 9d25a5a on mongoid:master.

3 similar comments
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.724% when pulling 52446aa on camiloforero:master into 9d25a5a on mongoid:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.724% when pulling 52446aa on camiloforero:master into 9d25a5a on mongoid:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.724% when pulling 52446aa on camiloforero:master into 9d25a5a on mongoid:master.

@dblock
Copy link
Collaborator

dblock commented Oct 22, 2018

I've seen similar failures elsewhere. I think newer mongodb/mongoid created a newer index version. I didn't get a chance to track that down, would be nice to pinpoint what exactly happen and I think the fix is just to change the specs.

@dblock
Copy link
Collaborator

dblock commented Nov 9, 2018

Do we want to resurrect this? You have my attention @camiloforero.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants