Skip to content

Add CI pipeline for unit tests#3

Merged
arielr-lt merged 6 commits into
mainfrom
feat/ci-pipeline-issue-70
Apr 17, 2026
Merged

Add CI pipeline for unit tests#3
arielr-lt merged 6 commits into
mainfrom
feat/ci-pipeline-issue-70

Conversation

@arielr-lt
Copy link
Copy Markdown
Collaborator

@arielr-lt arielr-lt commented Apr 13, 2026

Summary

  • Adds GitHub Actions CI workflow (.github/workflows/ci.yml) that runs unit tests on push and PR to main
  • Matrix across Python 3.10, 3.11, 3.12 with fail-fast: false
  • Fixes missing python-dotenv dev dependency used by tests/conftest.py

Closes CredentialEngine/ce-registry#70

Ariel Rolfo added 4 commits April 13, 2026 19:21
Adds GitHub Actions CI workflow that runs unit tests across Python
3.10/3.11/3.12 on push and PR to main. Also adds missing python-dotenv
dev dependency used by the test suite.

Closes #70
This is an internal tool — no need to test across multiple versions.
- Adds GitHub Actions release workflow triggered by version tags (v*.*.*)
- Switches version management to setuptools-scm (version derived from git tag)
- Adds CI-CD.md documenting how CI and the release process work
- Adds INSTALL.md with end-user installation instructions

Closes #70
Wires GitHub secrets as env vars so tests that depend on
CE_TEST_* values can run in CI.
Comment thread .github/workflows/ci.yml
Ariel Rolfo added 2 commits April 15, 2026 08:20
- Version format: YYYY.MM.DD.<run_number> (e.g. 2026.04.15.42)
- Both CI and Release pipelines now generate the version dynamically
  and pass it to the build via SETUPTOOLS_SCM_PRETEND_VERSION
- Release now triggered via workflow_dispatch instead of git tags
- CI now includes a build step to verify the package builds on every run
- Updates CI-CD.md to reflect the new release process
- Read __version__ via importlib.metadata at runtime so ce --version
  and ce version reflect the build-injected version
- Release pipeline now auto-triggers on push to main (pre-release)
  and supports manual workflow_dispatch for stable promotion
Copy link
Copy Markdown

@rohit-joy rohit-joy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! Approved.

@arielr-lt arielr-lt merged commit 600527d into main Apr 17, 2026
1 check failed
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.

2 participants