djorm-ext-pgfulltext icon indicating copy to clipboard operation
djorm-ext-pgfulltext copied to clipboard

Add hooks to run tests in travis-ci

Open troygrosfield opened this issue 10 years ago • 4 comments
trafficstars

Would be great to have more visibility to test output as well as transparency to new PRs ensuring tests all still pass. The task here would be to setup tests to run in travis-ci:

  • http://docs.travis-ci.com/user/languages/python/

@linuxlewis, you have to be the one to add the travis-ci service since you're the repo owner

  • https://github.com/linuxlewis/djorm-ext-pgfulltext/settings/hooks

This will also help test in other environments (such as django 1.8).

troygrosfield avatar Apr 27 '15 18:04 troygrosfield

@linuxlewis, I'd be happy to help get this all setup along with getting a valid release out to pypi. Feel free to add me as a collaborator on this project and/or as a releaser to pypi if you would like help. If for some reason you no longer wish to maintain this, I would suggest moving this to be a community own project at the following (org) url:

  • https://github.com/djorm-ext-pgfulltext/djorm-ext-pgfulltext

We just went through this with the django-ses app as well:

  • https://github.com/django-ses/django-ses/pull/68

That way we can get quicker turnaround time on issues that need to be resolved and more help from the community.

troygrosfield avatar Apr 27 '15 19:04 troygrosfield

Furthermore, this shows how to tests against a postgres instance in travis-ci:

  • http://docs.travis-ci.com/user/using-postgresql/

troygrosfield avatar Apr 27 '15 21:04 troygrosfield

Thanks for the suggestion. I started working on the travis integration in this PR: https://github.com/linuxlewis/djorm-ext-pgfulltext/pull/56

linuxlewis avatar May 16 '15 18:05 linuxlewis

Looks good. The other big issue that needs to be address is:

  • https://github.com/linuxlewis/djorm-ext-pgfulltext/issues/41

Currently, the management commands aren't in the released package in pypi.

troygrosfield avatar May 17 '15 01:05 troygrosfield