Skip to content

Use GitHub Actions#1

Draft
jmsmtn wants to merge 24 commits into
masterfrom
use-github-actions-for-tests-lint-checks
Draft

Use GitHub Actions#1
jmsmtn wants to merge 24 commits into
masterfrom
use-github-actions-for-tests-lint-checks

Conversation

@jmsmtn
Copy link
Copy Markdown
Owner

@jmsmtn jmsmtn commented May 9, 2026

Use GitHub Actions

Use GitHub Actions

While IdentityPython/pull/816 exists, it is fairly old. This PR tries to revive that effort in order to make this project easier to contribute to, maintain, and release.

Note: I'm not experienced with GitHub Actions - I mainly work with GitLab CI. But, I will bear any YAML burden here because CI is so important.

What your changes do and why you chose this solution

This PR introduces some GitHub Action workflows:

  • tests.yml - Run tests for Python versions 3.9 to 3.14. This doesn't use tox, but if needed I can easily change it to use tox.
  • checks.yml - for miscellaneous checks - right now to check that Poetry and it's lockfile is in a good state
  • lints.yml - Runs black, isort, and flake8.
    • I ran black and isort formats. The flake8 job will not fail, currently. It has some 200 items to resolve so I thought it would be best to make that a separate PR.
  • release-github.yml - Handle github releases
  • release-pypi.yml - Handle pypi and testpypi deployments. Tested with at Use GitHub Actions #1 and in test pypi and pypi

ACTIONS REQUIRED BY MAINTAINERS: In order to use the release-pypi.yml, trusted publishing must be setup in test.pypi.org and pypi.org. Having never done it myself, I thought that this was surprisingly easy to setup, but if it's too much right now, we can always remove the release-pypi.yml action.

Checklist

  • Checked that no other issues or pull requests exist for the same issue/change
  • Added tests covering the new functionality
    • n/a
  • Updated documentation OR the change is too minor to be documented
  • Updated CHANGELOG.md OR changes are insignificant

@jmsmtn jmsmtn changed the title Use GitHub Actions for tests, lints, and checks Use GitHub Actions May 9, 2026
Revert "Revert testing commits."

This reverts commit 63c83c2.
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.

1 participant