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

Packaging and Actions Part 1 #6

Merged
merged 36 commits into from
Aug 20, 2024
Merged

Packaging and Actions Part 1 #6

merged 36 commits into from
Aug 20, 2024

Conversation

acederberg
Copy link
Owner

General:

  • Moved installation to poetry.
  • Fixed mypy and ruff issues.
  • Refactoring of docker compose for CI setup.
  • Configuration modifications for CI tests.
  • model uuids now use UUID4.

Added actions for

  • Commit format.
  • Ruff and mypy checks.
  • Tests and Coverage with Pytest.
  • Publishing and Releases.

``app`` -> ``captura``
``client`` -> ``legere``
``dummy`` -> simulatus``.
Added initial actions.
Removed junk docs.
Fixed actual docs.
Added depends for nvim feedback.
So that important ``tests`` files are checked.
Separated linting and commit checks into separate jobs.
Added commitlintrc.
Renamed jobs.
Fixed matching mypy step names.
Fixed python version used.
Removed Helios Copyright (it is no longer verbatim his action code).
Better feedback in step summary.
It is not necessary to install ``sudo`` in the pipeline.
This is required by ``gh act``, which is a huge pain in the ass.
Also, found broken import of ``COUNT`` in ``test_views``. since ruff
ignores ``test_.*`` files.
Step summary is now markdown.
Added pytest step to ``commit_checks.yaml``.
Added test configuration.
Added configurable ``flakey.yaml`` path.
Fixed summary markdown.
Added \`.env\` to docker step.
[skip ci]
It had the wrong location.
Removed ``container_name`` from ``compose.yaml`` since it matches the
provided name.
Decided that working with volumes is a pain, decided to use ``extends``
to build off of base.
Added coverage reports.
Added compose down.
Updated coverage report configuration.
For some reason ``captura.models.uuid`` needed unique for more recent
MySQL images (8.2.x -> 8.4.x).
Wanted to do this later, but uuid collision occurred in pipeline.
Moved ci dependency installs into ``dockerfile``.
Added/corrected image labels in ``compose.yaml`` files and ``pyproject.toml``.
@acederberg acederberg merged commit 04e93e7 into master Aug 20, 2024
6 checks passed
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