search

honeybadger-io / honeybadger-python

Send Python and Django errors to Honeybadger.
https://www.honeybadger.io/
MIT License
15 stars 25 forks source link

Migrate from nose to tox/pytest #153

Open dotysan opened 1 year ago

dotysan commented 1 year ago

This is just a proof of concept...and an exercise in my learning how to use these tools.

Most notably, 2 Django and 5 Celery tests are still failing. But I temporarily disabled them just to prove it works.

A nice byproduct is that CI runs much quicker--and uses fewer minutes from GitHub Actions!

Thoughts?

dotysan commented 1 year ago

Closes #144.

dotysan commented 1 year ago

Closes #133.

dotysan commented 1 year ago

Closes #112.

subzero10 commented 1 year ago

Hey @dotysan, thank you for this work!

A nice byproduct is that CI runs much quicker--and uses fewer minutes from GitHub Actions! 🙌 🐎

I have a couple of questions:

  1. Are you working on making the failing tests pass (Django and Celery)? We need those passing in order to merge.
  2. Excuse my beginner question as I don't code Python often: Would this PR help us towards integrating with PDM or Poetry? Ideally, I would like us to use one of those tools for dependency management, testing and/or publishing.