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

Migrate from travis to GitHub actions & add mypy to the build #44

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

fuhrysteve
Copy link
Contributor

@fuhrysteve fuhrysteve commented Mar 10, 2021

Hi @cgoldberg!

Thought I'd stop by and offer some updates to the build. This includes:

  • Continued python 2.7 support
  • python 2.7 compatible typing
  • adds mypy in the build
  • adds a code coverage report to the build which reports on which lines are covered and which are not. Helpful little report if you want to see if new or modified code has actual test coverage
  • migrates from travis to github actions (travis has been problematic for a lot of maintainers lately)
  • Updates python 3 supported versions to currently supported versions (3.6-3.9)
  • adds codeql security scanning to the build (checks for common CVE vulnerabilities and will report on them in the security tab on GitHub)

Please also note that this PR will not trigger a build, because I'm not an owner of this repository. If you want to see evidence of a passing build, please see this PR on my own fork: fuhrysteve#1

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

Successfully merging this pull request may close these issues.

None yet

1 participant