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

Minor improvements and fixes #45

Merged
merged 4 commits into from
Feb 7, 2024
Merged

Conversation

bibipkins
Copy link
Owner

@bibipkins bibipkins commented Feb 7, 2024

  • minor code style improvements
  • set correct default value for show-failed-tests-only input
  • removed unnecessary test from test.yaml
  • fixed html test summary in case there are no tests to display

Copy link

github-actions bot commented Feb 7, 2024

Test - Results All

❌ Tests 82 / 88 (3 failed, 3 skipped) - failed in 20.1s
🔍 click here for more details

✏️ updated for commit 70e6e92

Copy link

github-actions bot commented Feb 7, 2024

Test - Results Success

✔️ Tests 44 / 44 - passed in 11.9s
🔍 click here for more details

✏️ updated for commit 70e6e92

Copy link

github-actions bot commented Feb 7, 2024

Test - Results Fail

❌ Tests 38 / 44 (3 failed, 3 skipped) - failed in 8.2s
🔍 click here for more details

✏️ updated for commit 70e6e92

Copy link

github-actions bot commented Feb 7, 2024

Test - Coverage OpenCover

✔️ Tests 44 / 44 - passed in 11.9s
✔️ Coverage 44.84% - passed with 44% threshold
📏 121 / 263 lines covered 🌿 5 / 18 branches covered
🔍 click here for more details

✏️ updated for commit 70e6e92

Copy link

github-actions bot commented Feb 7, 2024

Test - Coverage Cobertura

✔️ Tests 44 / 44 - passed in 11.9s
✔️ Coverage 44.84% - passed with 44% threshold
📏 121 / 263 lines covered 🌿 5 / 18 branches covered
🔍 click here for more details

✏️ updated for commit 70e6e92

Copy link

github-actions bot commented Feb 7, 2024

Test - Hide Success

❌ Tests 82 / 88 (3 failed, 3 skipped) - failed in 20.1s
🔍 click here for more details

✏️ updated for commit 70e6e92

Copy link

github-actions bot commented Feb 7, 2024

Test - Hide Output

❌ Tests 82 / 88 (3 failed, 3 skipped) - failed in 14.2s
🔍 click here for more details

✏️ updated for commit 70e6e92

Repository owner deleted a comment from github-actions bot Feb 7, 2024
Repository owner deleted a comment from github-actions bot Feb 7, 2024
Repository owner deleted a comment from github-actions bot Feb 7, 2024
Repository owner deleted a comment from github-actions bot Feb 7, 2024
Repository owner deleted a comment from github-actions bot Feb 7, 2024
Repository owner deleted a comment from github-actions bot Feb 7, 2024
Repository owner deleted a comment from github-actions bot Feb 7, 2024
@bibipkins bibipkins self-assigned this Feb 7, 2024
@bibipkins bibipkins linked an issue Feb 7, 2024 that may be closed by this pull request
@bibipkins bibipkins merged commit bd040a7 into main Feb 7, 2024
4 checks passed
@bibipkins bibipkins deleted the feature/minor-code-improvements branch February 7, 2024 16:51
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.

Large code base with hundreds of tests - summary missing
1 participant