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

update readme badge for testing #48

Merged
merged 1 commit into from
Apr 8, 2022
Merged

update readme badge for testing #48

merged 1 commit into from
Apr 8, 2022

Conversation

particledecay
Copy link
Owner

What? (description)

Use the coverage workflow since it is the only one run consistently from
the main branch, and also includes unit tests.

Why? (reasoning)

It's the workflow that's always run on main.

Screenshots (if applicable)

GitHub Issue (if applicable)

Acceptance

Check your PR for the following:

  • you included tests
  • you linted your code
  • your PR has only one commit (interactive rebase!)
  • your commit message follows Conventional Commit format
  • you are not reducing the total test coverage

Use the coverage workflow since it is the only one run consistently from
the main branch, and also includes unit tests.
@codeclimate
Copy link

codeclimate bot commented Apr 8, 2022

Code Climate has analyzed commit bb4e920 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 73.5% (0.0% change).

View more on Code Climate.

@particledecay particledecay merged commit a7dc76a into main Apr 8, 2022
@particledecay particledecay deleted the readme_test_badge branch April 8, 2022 00:29
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