Tigge / antfs-cli

Extracts FIT files from ANT-FS based sport watches such as Garmin Forerunner 60, 405CX, 310XT, 610 and 910XT.
MIT License
312 stars 76 forks source link

Use setuptools and add travisCI and Coveralls.io #119

Closed Toilal closed 9 years ago

Toilal commented 9 years ago

Currently there's no test in this project, but is here the structure with setuptools and travisCI configuration.

Toilal commented 9 years ago

antfs-cli module should be renamed => antfs_cli, or maybe even simpler antfs. see https://travis-ci.org/Tigge/Garmin-Forerunner-610-Extractor/jobs/45695612

As antfs is already registered on pypi, i would suggest antfs-cli as pypi package name and repository name and antfs_clias module name (antfs_cli.py filename). Are you ok ?

Tigge commented 9 years ago

Had a bit of power outage here. Yeah, antfs_cli is fine by me.

Tigge commented 9 years ago

Should this still fail now?

Toilal commented 9 years ago

I'll run it again, it should work now.

Toilal commented 9 years ago

Well i can't run it because i don't have access to the repository, you can restart the run in travisCI page. Click on the Details link on the right just here, and then restart the build.

Toilal commented 9 years ago

I'll rebase on split branch, and it will relaunch build on travisCI

Tigge commented 9 years ago

I restarted the build but it still seems to fail to find the openant library.

Tigge commented 9 years ago

Eventually openant should be published to PyPi, but for now it should be possible for TravisCI to fetch it directly from the git repository, right?

Tigge commented 9 years ago

Adding -e git+https://github.com/Tigge/openant.git#egg=openant in requirements.txt perhaps?

Toilal commented 9 years ago

That's what i tried to do in travis.yml, but seems it doesn't work, i'll modify requirements.txt to try.

coveralls commented 9 years ago

Coverage Status

Changes Unknown when pulling b30f987e75fa398a63c43c74e4525831109c108e on Toilal:ci into \ on Tigge:split**.

Tigge commented 9 years ago

Nice!

Toilal commented 9 years ago

I can rebase in a single commit if you want ?

Tigge commented 9 years ago

Yes, please do-

coveralls commented 9 years ago

Coverage Status

Changes Unknown when pulling d92db2ec93ff31d82e444e4753eb5c5f0ab1caca on Toilal:ci into \ on Tigge:split**.

Tigge commented 9 years ago

Thanks!