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.

Sign up for GitHub

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

Jump to bottom

more test coverage #5

Open
SchoolGuy opened this issue Dec 7, 2021 · 2 comments
Open

more test coverage #5

SchoolGuy opened this issue Dec 7, 2021 · 2 comments

Comments

@SchoolGuy
Copy link
Contributor

This is a card which I created from the TODO file bullet point list.
SchoolGuy added a commit to SchoolGuy/node2rpm that referenced this issue Dec 7, 2021
@SchoolGuy
Remove TODO file
68b7969 
Since GitHub issues openSUSE-zh#2, openSUSE-zh#3, openSUSE-zh#4 & openSUSE-zh#5 contain the information we can
remove the TODO file.
@SchoolGuy SchoolGuy mentioned this issue Dec 7, 2021
Merged
@SchoolGuy
Copy link
Contributor Author

I would say we can use the built-in tool for this and use Codecov as a nice overview for this: https://about.codecov.io/blog/getting-started-with-code-coverage-for-golang/

I had in the past a good experience with Codecov for the following reasons:

  • Low maintenance after the initial setup.
  • Minimalistic UI
  • Coverage History so we can see how the Coverage develops.
  • Coverage reports on PRs.

@SchoolGuy
Copy link
Contributor Author

P.S.: In the blogpost they also show how to upload the report, so the setup is also quite easy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@SchoolGuy