Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Testing and building #163

Closed
wants to merge 1 commit into from
Closed

Testing and building #163

wants to merge 1 commit into from

Conversation

christopherpickering
Copy link

Summary

This pr adds a few things to make building/testing better:

  • added coverage
  • added github lint workflow (uses mypy, black, ruff. more ruff codes should be enabled/fixed in the future.. trying to keep the pr small)
  • change packager to poetry - much easier to releae and also manage the dev deps for all the backends
  • added tox to run the tests
  • added github test workflow
  • added basic pre-commit file

After merging this we can rebase several of the other pr's and merge them with a few minor tweaks.

Using this:

Installing for dev:
poetry install

Running tests:
poetry run tox

Running for formatting & linting
poetry run tox -e isort,black,lint

@christopherpickering
Copy link
Author

The failed tests are valid failures in the current code and either have or should have other pr's to fix them.

@christopherpickering
Copy link
Author

moved to #170

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant