-
Notifications
You must be signed in to change notification settings - Fork 3
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
ci: initial test pipeline UI #109
Conversation
Code Coverage Report
|
@fbufler @FlorianZieglerBosch I think this is rather a general issue with the test execution than actual errors |
@fbufler what do you mean , you run it inside the folder in your screen capture, not? |
Yes I do. cd yaku-ui (Unsure if it works without the build as I cant really test it well) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure but isn't the build part missing?
We stop at testing but based on the workflow name I would assume that we also run npm run build.
If that's not the case LGTM
npm run build is done in the docker build here , I can refactor this later |
fixes #107