Skip to content

disable timeutils test to prevent depending on mgo#85

Merged
araddon merged 1 commit intoaraddon:masterfrom
egonelbre:remove-mgo
May 10, 2019
Merged

disable timeutils test to prevent depending on mgo#85
araddon merged 1 commit intoaraddon:masterfrom
egonelbre:remove-mgo

Conversation

@egonelbre
Copy link
Copy Markdown
Contributor

Currently timeutils benchmark ends up pulling in mgo as a dependency to other projects. Comment out the test so it's not a strict requirement.

Unfortunately, didn't find a nice way to avoid creating a dependency and still make benchmarks work with a special flag.

@codecov-io
Copy link
Copy Markdown

codecov-io commented May 10, 2019

Codecov Report

Merging #85 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #85   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines         885    885           
=====================================
  Hits          885    885

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0d74ffc...ad79b62. Read the comment docs.

@araddon
Copy link
Copy Markdown
Owner

araddon commented May 10, 2019

awesome, ty

@araddon araddon merged commit d2ba703 into araddon:master May 10, 2019
@egonelbre egonelbre deleted the remove-mgo branch May 10, 2019 22:39
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.

3 participants