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

Development #12

Merged
merged 32 commits into from
Sep 4, 2023
Merged

Development #12

merged 32 commits into from
Sep 4, 2023

Conversation

ThomasJunk
Copy link
Contributor

These commits form the current state of development.
To mitigate bigger drift between main and development we decided to get both in sync.

Thomas Junk added 30 commits August 23, 2023 12:27
@github-actions
Copy link

github-actions bot commented Sep 4, 2023

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 88.23% / 60% 165 / 187
🟢 Statements 87.92% / 60% 182 / 207
🟢 Functions 95.23% / 60% 40 / 42
🟢 Branches 82.2% / 60% 97 / 118
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/lib/singleview/general/docmodel.ts 83.72% 83.33% 95.45% 83.33% 75, 77, 79, 91, 95, 139, 144-149, 146, 146-147, 147-148, 150
src/lib/singleview/general/docmodeltypes.ts 100% 100% 100% 100%
src/lib/singleview/productvulnerabilities/productvulnerabilities.ts 90.35% 80.76% 95% 90.74% 27, 27-28, 48, 63, 83-84, 110-115, 111-114, 168-172
src/lib/singleview/productvulnerabilities/productvulnerabilitiestypes.ts 100% 100% 100% 100%
Generated in workflow #3

s-l-teichmann
s-l-teichmann previously approved these changes Sep 4, 2023
Copy link
Contributor

@s-l-teichmann s-l-teichmann left a comment

Choose a reason for hiding this comment

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

As it is work in progress it looks good to me.

Copy link
Contributor

@JanHoefelmeyer JanHoefelmeyer left a comment

Choose a reason for hiding this comment

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

LGTM

@ThomasJunk ThomasJunk merged commit cb08c05 into main Sep 4, 2023
3 checks passed
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.

3 participants