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

#10 Add test coverage report #12

Merged
merged 4 commits into from
Sep 30, 2023
Merged

Conversation

nevendyulgerov
Copy link
Contributor

No description provided.

@nevendyulgerov nevendyulgerov added the Type: Chore Improvements to the code base and DX. It has no impact on external users e.g. Update deps label Sep 29, 2023
@nevendyulgerov nevendyulgerov self-assigned this Sep 29, 2023
@@ -32,6 +32,12 @@ jobs:
- name: Run tests
run: npm run ci:test

- name: Publish coveralls report
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had to add the coveralls build step to the docker.yaml file as well, because right now we're using it to build the app.

@nevendyulgerov nevendyulgerov marked this pull request as ready for review September 29, 2023 09:51
@nevendyulgerov nevendyulgerov merged commit 15580dd into main Sep 30, 2023
1 check passed
@nevendyulgerov nevendyulgerov deleted the chore/10-add-test-coverage-report branch September 30, 2023 16:02
@nevendyulgerov nevendyulgerov linked an issue Sep 30, 2023 that may be closed by this pull request
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Chore Improvements to the code base and DX. It has no impact on external users e.g. Update deps
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Chore: Publish test coverage report
2 participants