networkx / networkx-metis

NetworkX Addon to allow graph partitioning with METIS
Other
78 stars 21 forks source link

Set up Travis CI #12

Closed ysitu closed 9 years ago

ysitu commented 9 years ago

Run nose and pylint; coverage is optional.

OrkoHunter commented 9 years ago

Can you please adjust the due date of the milestone with the release of NetworkX 2.0?

ysitu commented 9 years ago

I have changed the milestone for networkx-metis 1.0 to Jul 8. I am thinking that maybe we should cherrypick networkx/networkx#1595 into the 1.10 so that we do not have to wait till release of networkx 2.0, which will be after end of GSoC, before releasing networkx-metis.

chebee7i commented 9 years ago

I'd be okay with that! It is a minimal change to NetworkX.

OrkoHunter commented 9 years ago

Do we have to run pylint in the script of .travis.yml ?

OrkoHunter commented 9 years ago

If not, can we let #16 in and one of the collaborators set up the hook on travis-ci.org ?

ysitu commented 9 years ago

We should run pylint. It catches more than style errors.

OrkoHunter commented 9 years ago

Okay... All I need is just to add pylint * in the script section?

ysitu commented 9 years ago

Use pylint networkx.addons.metis.

OrkoHunter commented 9 years ago

@dschult @jtorrents @hagberg May one of you please set up the hook on travis-ci.org for this repository?