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

add coverage threshold #15

Merged
merged 1 commit into from
Jan 8, 2024

Conversation

DanielMurray97
Copy link
Contributor

@DanielMurray97 DanielMurray97 commented Jan 8, 2024

JIRA link

NTRNL-282

Description

Add coverage threshold to prepush and github action

Work checklist

  • Tests added where applicable
  • No vulnerability added

@DanielMurray97 DanielMurray97 force-pushed the ntrnl-282-add-coverage-threshold-to-api-sdk branch from 5c29768 to 1b5ec57 Compare January 8, 2024 13:41
@DanielMurray97 DanielMurray97 force-pushed the ntrnl-282-add-coverage-threshold-to-api-sdk branch from 1b5ec57 to 0a715c2 Compare January 8, 2024 13:42
@DanielMurray97 DanielMurray97 reopened this Jan 8, 2024
Copy link

github-actions bot commented Jan 8, 2024

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 97.06% 66/68
🟢 Branches 83.33% 20/24
🟢 Functions 96.55% 28/29
🟢 Lines 96.43% 54/56

Test suite run success

19 tests passing in 4 suites.

Report generated by 🧪jest coverage report action from bc3e0a1

@DanielMurray97 DanielMurray97 marked this pull request as ready for review January 8, 2024 14:25
Copy link
Contributor

@Mouhajer-CO Mouhajer-CO left a comment

Choose a reason for hiding this comment

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

🚀

@DanielMurray97 DanielMurray97 merged commit 9074ecb into main Jan 8, 2024
7 checks passed
@DanielMurray97 DanielMurray97 deleted the ntrnl-282-add-coverage-threshold-to-api-sdk branch January 8, 2024 14:46
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.

2 participants