Skip to content

feat: add 'ignore for this file' in notifications#805

Draft
Strum355 wants to merge 1 commit intofabric8-analytics:mainfrom
Strum355:nsc/notification-dontshow
Draft

feat: add 'ignore for this file' in notifications#805
Strum355 wants to merge 1 commit intofabric8-analytics:mainfrom
Strum355:nsc/notification-dontshow

Conversation

@Strum355
Copy link
Copy Markdown
Collaborator

@Strum355 Strum355 commented Jul 21, 2025

Adds an "Ignore this file" button to notifications from component analysis errors or vulns discovered during component analysis, using user-configurable exclude patterns from #803

image

Current design of the status bar doesn't allow resetting (in general the status bar needs a rework)

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 21, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 8.69565% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 60.68%. Comparing base (dcf9458) to head (0068492).

Files with missing lines Patch % Lines
src/extension.ts 5.26% 18 Missing ⚠️
src/config.ts 0.00% 3 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #805      +/-   ##
==========================================
- Coverage   61.13%   60.68%   -0.45%     
==========================================
  Files          31       31              
  Lines        1446     1460      +14     
  Branches      268      271       +3     
==========================================
+ Hits          884      886       +2     
- Misses        547      559      +12     
  Partials       15       15              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Strum355 Strum355 force-pushed the nsc/notification-dontshow branch from 418166a to d5ea5c7 Compare July 21, 2025 16:17
@Strum355 Strum355 force-pushed the nsc/notification-dontshow branch from 5793163 to 0068492 Compare January 5, 2026 17:31
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