-
Notifications
You must be signed in to change notification settings - Fork 160
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
fea: transform alert from enum lib into diagnostic #1579
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1579 +/- ##
==========================================
- Coverage 78.70% 78.63% -0.07%
==========================================
Files 219 219
Lines 6946 6952 +6
==========================================
Hits 5467 5467
- Misses 1282 1288 +6
Partials 197 197
|
35ffc8a
to
9556ea3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 non-blocking notes, I'll approve in advance π
Before we merge this, I/you/we could test this branch out in snyk-iac-test and see if we can spike out bubbling up these non-fatal errors from diagnostics. WDYT?
100% in sync with that, let's do a small pairing session on that, shoot me an invite ππ» |
9556ea3
to
c5eec8b
Compare
c5eec8b
to
a1a4d70
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
1ad43c0
to
de753eb
Compare
Description
transform alert from enum lib into diagnostic when using the facade implementing the new api