-
Notifications
You must be signed in to change notification settings - Fork 27
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
Set CI pipeline with github actions #14
Comments
I see no mean to add an action at https://github.com/apssouza22/chatflow/actions Apparently, it's so because I have no admin rights. |
@vporton It is done through code. So you would do it to your repo, once it works fine you can open a normal PR |
@apssouza22 Probably you have an idea which of Python actions to use: |
@vporton it should be docker related action instead of python. The Required python will be in the docker image |
See the picture above: There are several docker images for Python. Maybe, you know which one to choose? @apssouza22 |
Test python application might work |
Python does not provide test mode for an application only to compile and not start. So, we can test Python only by writing tests (I mean |
Having the build and the start now would already bring some confidence.
Than we can add integration tests to call the API.
We can start with the integration test as well. Either way is fine.
We want something similar to this
https://dev.to/sahanonp/setup-docker-for-integration-testing-in-github-action-39fn
…On Fri 13 Oct 2023, 02:25 mathematician, ***@***.***> wrote:
Python does not provide test mode for an application only to compile and
not start.
So, we can test Python only by writing tests (I mean pytest or like
this). This issue should be postponed to be fulfilled only after we have a
testing framework.
—
Reply to this email directly, view it on GitHub
<#14 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAC3RWMTAVLIWVK6SKCCYADX7CJ73ANCNFSM6AAAAAA32LV454>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Build the docker imagine and run the service to later add integration tests
The text was updated successfully, but these errors were encountered: