Run offline tests on every push and every pull request

[oshoma]

Your checklist for this pull request

Thank you for submitting a pull request! To speed up the review process, please follow this checklist:

• My Pull Request is small and focused on one topic so it can be reviewed easily
• My code follows the style guidelines of this project (make format)
• Commit messages are detailed
• I have performed a self-review of my code
• I commented hard-to-understand parts of my code
• I updated the documentation (docstrings, /docs)
• My changes generate no new warnings (or explain any new warnings and why they're ok)
• I have added tests that prove my fix is effective or that my feature works
• All tests pass when I run pytest tests (offline mode)

Additional steps for code with networking dependencies:

• I followed the offline and online testing guidelines
• All tests pass when I run pytest tests --external_api (online mode, making network calls)

Description

Previously we would just run tests on pull requests against main. GitHub Actions usage is free for standard GitHub-hosted runners in public repositories, so we might as well test more comprehensively.

💔Thank you!

[20001LastOrder]

Should we restrict the push to only the main branch?

[oshoma]

@20001LastOrder I think testing on all branches is helpful.

If this was a private repo I would be more sparing, but in a public repo I think it's nice if all contributors can see test results quickly.

[20001LastOrder]

LGTM