-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Java: add diagnostic query indicating low database quality #16810
Java: add diagnostic query indicating low database quality #16810
Conversation
db4ea46
to
884bd20
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.
@smowton Some small style things in the warning text but overall looks good from a docs perspective!
ad34a58
to
51c21b7
Compare
Here's the TSP updated with the latest text: https://github.com/smowton/test-diagnostics-from-ql/security/code-scanning/tools/CodeQL/status/configurations/api/0a0e717e7702ad6128802b437a72832333465cec67163ebe61b8292092f457c4 |
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.
This makes a lot of sense and will be really useful. I do have a question about the change in the workflows.
Also, does this affect C# as well? Should we make the same change for that language, too? |
It does, but we just shipped to public beta to C# and we want to wait a little while to look at the data to see where we put the line for 🟡 / 🟢 |
Green is meant to indicate that the quality of the database is close to a traced database. Yellow is meant to indicate that it is not. For both Java and C#, 95% seems to be a sensible threshold based on that principle. This may result in more % of databases being flagged for C#, but that makes sense, as it's consistent with the data. |
d8e7874
to
9be2c1b
Compare
Co-authored-by: Sarita Iyer <66540150+saritai@users.noreply.github.com> Co-authored-by: Chad Bentz <1760475+felickz@users.noreply.github.com>
9be2c1b
to
5cee392
Compare
No description provided.