search

jpfairbanks / GraphMatrices.jl

A Julia package for strongly typed graph matrices.
Other
6 stars 5 forks source link

Dependencies #15

Open sbromberger opened 8 years ago

sbromberger commented 8 years ago

Hey, a purely selfish request:

Currently, in REQUIRE, you have

  julia 0.3
  FactCheck
  Compat
  Docile

Can you rewrite your tests to use Base.Test instead of FactCheck? If so, you can probably get rid of all dependencies, making this a leaf package, which allows us to use it natively in LightGraphs :)

jpfairbanks commented 8 years ago

FactCheck is a JuliaLang core repo. Doesn't that allow LG to depend on it? If not, then I blame myself for not paying attention to that policy when I voted for it. Also it excludes any repo that uses Compat.

sbromberger commented 8 years ago

It is - but we should standardize on a single testing framework for LG. I'm ok with using either Base.Test or Base.FactCheck, but we need to convert code in either case. What would you prefer?