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

chore: remove qualys parser #350

Merged
merged 5 commits into from
Oct 9, 2023

Conversation

sozercan
Copy link
Member

@sozercan sozercan commented Oct 5, 2023

Describe the changes in this pull request using active verbs such as Add, Remove, Replace ...

Remove Qualys parser as we don't have access or any tests for this

Closes #<issue_ID>

Signed-off-by: Sertac Ozercan <[email protected]>
@sozercan sozercan changed the title remove qualys parser chore: remove qualys parser Oct 5, 2023
@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (0237956) 31.77% compared to head (4ed8afc) 32.54%.
Report is 2 commits behind head on main.

❗ Current head 4ed8afc differs from pull request most recent head 0d746a8. Consider uploading reports for the commit 0d746a8 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #350      +/-   ##
==========================================
+ Coverage   31.77%   32.54%   +0.76%     
==========================================
  Files          18       17       -1     
  Lines        1630     1555      -75     
==========================================
- Hits          518      506      -12     
+ Misses       1081     1020      -61     
+ Partials       31       29       -2     
Files Coverage Δ
pkg/report/report.go 73.33% <ø> (-1.67%) ⬇️
pkg/patch/patch.go 6.36% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Sertac Ozercan <[email protected]>
@sozercan sozercan force-pushed the remove-qualys-scanner branch from 1ed721a to a4eda0f Compare October 6, 2023 20:02
@sozercan sozercan requested a review from ashnamehrotra October 9, 2023 16:34
@sozercan sozercan merged commit 462fb5a into project-copacetic:main Oct 9, 2023
8 checks passed
@sozercan sozercan deleted the remove-qualys-scanner branch October 9, 2023 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants