Skip to content

Docker workflow #53

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

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Docker workflow #53

wants to merge 17 commits into from

Conversation

Hevagog
Copy link
Collaborator

@Hevagog Hevagog commented May 16, 2025

Docker workflow

Description

Added docker compose build and run to the actions to further validate changes.
For now it only runs containers and doesn't run additional scripts

  • Type of Change:
    • New feature
    • Bug fix
    • Code refactoring
    • Documentation update
    • Other (please describe):

How Has This Been Tested?

Describe the tests that you ran to verify your changes. Include details on the testing environment, test cases, and any relevant configurations. If applicable, provide steps to reproduce the testing process.

  • Unit tests
  • Integration tests
  • Manual tests
  • Skip test - Please provide an explanation why you skip tests

We encourage you to keep the code coverage percentage at 90% and above.

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

Checklist

Ensure that all the following tasks are completed before submitting your PR:

  • Code is formatted and has been linted using 'makefile' run-check option
  • New and existing tests pass locally.
  • All relevant documentation has been updated (e.g., docstrings, README, etc.).
  • Dependencies have been updated in pyproject.toml, and uv.lock has been regenerated accordingly.
  • This PR has been reviewed by at least one other developer.

@Hevagog Hevagog requested a review from dJaniga May 16, 2025 22:10
@Hevagog Hevagog marked this pull request as ready for review May 23, 2025 20:18
@Hevagog
Copy link
Collaborator Author

Hevagog commented May 23, 2025

@dJaniga ready for review

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