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
filtering comment findings now, still not editing existing comment #524
Conversation
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.
Looks very solid. Just nuke the build
folder and this should be good to go!
* | ||
*/ | ||
const types_1 = require("./types"); | ||
function sortBySeverity(a, b) { |
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.
nuke this file please (the whole build
folder)
const dummyObject = {}; | ||
|
||
const v8Handler = Error.prepareStackTrace; | ||
Error.prepareStackTrace = function (dummyObject, v8StackTrace) { |
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.
Nit that we should add to the linter: If you don't use an anonymous function, it should be named.
@@ -4,7 +4,7 @@ | |||
"private": true, | |||
"dependencies": { | |||
"@aws-sdk/client-s3": "^3.52.0", | |||
"@lunatrace/lunatrace-common": "link:../common", | |||
"@lunatrace/lunatrace-common": "workspace:~", |
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.
the linter seems to be complaining about this. Are you sure this is the correct syntax?
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.
We are now using Reviews in "comment" mode instead of straight comments. They are updated instead of duplicated.
@freeqaz I rolled back most of the changes you made when you copy pasted the code, and then for the backend used hasura's return data from the
InsertScan
mutation instead of the input data because the shape was much more similar to what the frontend was seeing, and we could request IgnoredVulnerabilities simultaneously which we needed.